fix email notifications when configured by env var

cdzombak · Nov 28, 2023 · cef9b21 · cef9b21
1 parent 665aaf7
commit cef9b21
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/main.go b/main.go
@@ -276,7 +276,6 @@ func main() {
 		mailCfg.mailFrom = os.Getenv(MailFromEnvVar)
 	}
 	if mailCfg.mailFrom == "" {
-		// TODO(cdzombak):
 		mailCfg.mailFrom = "runner@" + hostname
 	}
 	if mailCfg.smtpUser == "" {
@@ -299,7 +298,7 @@ func main() {
 		}
 	}
 	if mailCfg.mailTo != "" && strings.Contains(mailCfg.mailTo, "@") {
-		if *smtpUser != "" && *smtpPass != "" && *smtpHost != "" {
+		if mailCfg.smtpUser != "" && mailCfg.smtpPassword != "" && mailCfg.smtpHost != "" {
 			shouldMailOutput = true
 
 			if mailCfg.smtpPort < 1 || mailCfg.smtpPort > 65535 {