Fix blackfriday-slack panics on lists (#980)

* fix: blackfriday-slack panics on lists

* Make panic in notifier logged as error

notifier/notifier.go

@@ -177,7 +177,7 @@ func (notifier *StandardNotifier) reschedule(pkg *NotificationPackage, reason st
 func (notifier *StandardNotifier) runSender(sender moira.Sender, ch chan NotificationPackage) {
 	defer func() {
 		if err := recover(); err != nil {
-			notifier.logger.Warning().
+			notifier.logger.Error().
 				String(moira.LogFieldNameStackTrace, string(debug.Stack())).
 				Interface("recovered_err", err).
 				Msg("Notifier panicked")

senders/slack/slack.go

@@ -10,7 +10,6 @@ import (
 	slackdown "github.com/moira-alert/blackfriday-slack"
 	"github.com/moira-alert/moira"
 	"github.com/moira-alert/moira/senders"
-	blackfriday "github.com/russross/blackfriday/v2"
 
 	slack_client "github.com/slack-go/slack"
 )
@@ -134,7 +133,7 @@ func (sender *Sender) buildMessage(events moira.NotificationEvents, trigger moir
 func (sender *Sender) buildDescription(trigger moira.TriggerData) string {
 	desc := trigger.Desc
 	if trigger.Desc != "" {
-		desc = string(blackfriday.Run([]byte(desc), blackfriday.WithRenderer(&slackdown.Renderer{})))
+		desc = string(slackdown.Run([]byte(desc)))
 		desc += "\n"
 	}
 	return desc

senders/slack/slack_test.go

@@ -236,4 +236,23 @@ some other text italic text
 			So(actual, ShouldResemble, expected)
 		})
 	})
+
+	Convey("Build desc with lists", t, func() {
+		trigger := moira.TriggerData{
+			Desc: `
+1. a
+`,
+		}
+
+		expected := ` 1. a
+
+
+`
+
+		Convey("Expect buildDescription not to panic", func() {
+			actual := sender.buildDescription(trigger)
+
+			So(actual, ShouldEqual, expected)
+		})
+	})
 }