fix bug determining whether to send emails

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