Merge pull request linkedin#125 from exponea/fix-config-notify-interval

Fix handling of Notify.Interval default
lconnell · Aug 26, 2016 · 11bb2b6 · 11bb2b6
2 parents ff0c188 + 6dd9a52
commit 11bb2b6
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/config.go b/config.go
@@ -328,6 +328,11 @@ func ValidateConfig(app *ApplicationContext) error {
 		}
 	}
 
+	// Notify
+	if app.Config.Notify.Interval == 0 {
+		app.Config.Notify.Interval = 10
+	}
+
 	// SMTP server config
 	if app.Config.Smtp.Server != "" {
 		if !validateHostname(app.Config.Smtp.Server) {
@@ -356,11 +361,6 @@ func ValidateConfig(app *ApplicationContext) error {
 		}
 		// Username and password are not validated - they're optional
 
-		// Notify
-		if app.Config.Notify.Interval == 0 {
-			app.Config.Notify.Interval = 10
-		}
-
 		// Email configs
 		for email, cfg := range app.Config.Emailnotifier {
 			if !validateEmail(email) {