Browse Source

fix(alerting): fixed email notification so it splits email address multiple ways, fixes #6353

Torkel Ödegaard 9 years ago
parent
commit
4999fff29b
1 changed files with 10 additions and 1 deletions
  1. 10 1
      pkg/services/alerting/notifiers/email.go

+ 10 - 1
pkg/services/alerting/notifiers/email.go

@@ -29,9 +29,18 @@ func NewEmailNotifier(model *m.AlertNotification) (alerting.Notifier, error) {
 		return nil, alerting.ValidationError{Reason: "Could not find addresses in settings"}
 	}
 
+	// split addresses with a few different ways
+	addresses := strings.FieldsFunc(addressesString, func(r rune) bool {
+		switch r {
+		case ',', ';', '\n':
+			return true
+		}
+		return false
+	})
+
 	return &EmailNotifier{
 		NotifierBase: NewNotifierBase(model.Id, model.IsDefault, model.Name, model.Type, model.Settings),
-		Addresses:    strings.Split(addressesString, `;`),
+		Addresses:    addresses,
 		log:          log.New("alerting.notifier.email"),
 	}, nil
 }