Fix wrong SMTP port when TLS is being used (fixes #3574) (#3607)
authorManuel Bleichner <mbleichner@gmail.com>
Thu, 13 Jul 2023 11:10:14 +0000 (13:10 +0200)
committerGitHub <noreply@github.com>
Thu, 13 Jul 2023 11:10:14 +0000 (13:10 +0200)
Co-authored-by: Manuel Bleichner <manuel.bleichner@bvu.de>
crates/utils/src/email.rs

index 3c8d7a1a126780f919998a23f8ee2972a9fa412b..fdff19033d7998914addd6a7c9c84da50afd23c8 100644 (file)
@@ -68,12 +68,10 @@ pub async fn send_email(
   // is bad.
 
   // Set the TLS
-  let builder_dangerous = AsyncSmtpTransport::builder_dangerous(smtp_server).port(smtp_port);
-
   let mut builder = match email_config.tls_type.as_str() {
-    "starttls" => AsyncSmtpTransport::starttls_relay(smtp_server)?,
-    "tls" => AsyncSmtpTransport::relay(smtp_server)?,
-    _ => builder_dangerous,
+    "starttls" => AsyncSmtpTransport::starttls_relay(smtp_server)?.port(smtp_port),
+    "tls" => AsyncSmtpTransport::relay(smtp_server)?.port(smtp_port),
+    _ => AsyncSmtpTransport::builder_dangerous(smtp_server).port(smtp_port),
   };
 
   // Set the creds if they exist