Skip to content

[N001] Use a non-TLS client for SMTP on port 25#5238

Merged
TD-er merged 6 commits intoletscontrolit:megafrom
tonhuisman:bugfix/N001-use-SMTP-for-port-25
Feb 1, 2025
Merged

[N001] Use a non-TLS client for SMTP on port 25#5238
TD-er merged 6 commits intoletscontrolit:megafrom
tonhuisman:bugfix/N001-use-SMTP-for-port-25

Conversation

@tonhuisman
Copy link
Contributor

@tonhuisman tonhuisman commented Jan 31, 2025

Resolves #5237

Bugfix:

  • Even when using port 25 a secure client was started for the mail-server connection. Now when using port 25, or port 2525, a default WiFiClient is used, instead of a secure client.
  • Close & destroy client also when server-connection (login?/network?) fails.
  • Documentation updated about the use of ports 25 and 2525 for ESP32 builds with TLS included.

TODO:

@TD-er TD-er merged commit 45c4d74 into letscontrolit:mega Feb 1, 2025
168 checks passed
@tonhuisman tonhuisman deleted the bugfix/N001-use-SMTP-for-port-25 branch February 2, 2025 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FR] Make it optional to use SMTPS for email notifications on ESP32 devices.

2 participants