Skip to content

Commit ea24560

Browse files
authored
Don't require password for SMTP connection (#739) (#740)
* don't require password * only set when password
1 parent 08400b2 commit ea24560

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

server/BudgetBoard.WebAPI/Utils/EmailSender.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,6 @@ public async Task SendEmailAsync(string email, string subject, string htmlMessag
2424
}
2525

2626
var senderPassword = Configuration.GetValue<string>("EMAIL_SENDER_PASSWORD");
27-
if (string.IsNullOrEmpty(senderPassword))
28-
{
29-
throw new ArgumentNullException(nameof(senderPassword));
30-
}
3127

3228
var smtpHost = Configuration.GetValue<string>("EMAIL_SMTP_HOST");
3329
if (string.IsNullOrEmpty(smtpHost))
@@ -48,9 +44,13 @@ public async Task SendEmailAsync(string email, string subject, string htmlMessag
4844
EnableSsl = true,
4945
UseDefaultCredentials = false,
5046
Port = smtpPort,
51-
Credentials = new NetworkCredential(senderUsername, senderPassword),
5247
};
5348

49+
if (!string.IsNullOrEmpty(senderPassword))
50+
{
51+
smtp.Credentials = new NetworkCredential(senderUsername, senderPassword);
52+
}
53+
5454
await smtp.SendMailAsync(mm);
5555
}
5656
}

0 commit comments

Comments
 (0)