hrynco/hrynco-notification-service
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: correct smtp ssl negotiation mode per port

Browse Source 
- Port 465: SslOnConnect (implicit SSL)
- Port 587 + UseSsl: StartTls (STARTTLS upgrade)
- UseSsl=false: None

Ref: IT-628

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
Anatolii Grynchuk
2026-05-02 03:45:59 +03:00
parent 6434e3e636
commit f7c35671b7
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
HrynCo.NotificationService.Web/Controllers/Admin/AdminChannelsController.cs
+5 -1
View File
@@ -163,7 +163,11 @@ public class AdminChannelsController : Controller
}; };
using var client = new SmtpClient(); using var client = new SmtpClient();
var secureSocket = smtp.UseSsl ? SecureSocketOptions.SslOnConnect : SecureSocketOptions.StartTlsWhenAvailable; var secureSocket = smtp.Port == 465
? SecureSocketOptions.SslOnConnect
: smtp.UseSsl
? SecureSocketOptions.StartTls
: SecureSocketOptions.None;
await client.ConnectAsync(smtp.Host, smtp.Port, secureSocket, ct); await client.ConnectAsync(smtp.Host, smtp.Port, secureSocket, ct);
if (!string.IsNullOrWhiteSpace(smtp.Username)) if (!string.IsNullOrWhiteSpace(smtp.Username))