瀏覽代碼

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

Kailash Nadh 3 年之前
父節點
當前提交
51da1a16a0
共有 1 個文件被更改,包括 4 次插入0 次删除
  1. 4 0
      cmd/notifications.go

+ 4 - 0
cmd/notifications.go

@@ -22,6 +22,10 @@ type notifData struct {
 
 
 // sendNotification sends out an e-mail notification to admins.
 // sendNotification sends out an e-mail notification to admins.
 func (app *App) sendNotification(toEmails []string, subject, tplName string, data interface{}) error {
 func (app *App) sendNotification(toEmails []string, subject, tplName string, data interface{}) error {
+	if len(toEmails) == 0 {
+		return nil
+	}
+
 	var b bytes.Buffer
 	var b bytes.Buffer
 	if err := app.notifTpls.ExecuteTemplate(&b, tplName, data); err != nil {
 	if err := app.notifTpls.ExecuteTemplate(&b, tplName, data); err != nil {
 		app.log.Printf("error compiling notification template '%s': %v", tplName, err)
 		app.log.Printf("error compiling notification template '%s': %v", tplName, err)