Add check to skip admin notifications with no e-mails. Closes #300.

knadh · Sep 24, 2021 · 51da1a1 · 51da1a1
1 parent c2a3f7d
commit 51da1a1
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/cmd/notifications.go b/cmd/notifications.go
@@ -22,6 +22,10 @@ type notifData struct {
 
 // sendNotification sends out an e-mail notification to admins.
 func (app *App) sendNotification(toEmails []string, subject, tplName string, data interface{}) error {
+	if len(toEmails) == 0 {
+		return nil
+	}
+
 	var b bytes.Buffer
 	if err := app.notifTpls.ExecuteTemplate(&b, tplName, data); err != nil {
 		app.log.Printf("error compiling notification template '%s': %v", tplName, err)