[PR #970] [MERGED] server: clarify SMTP error message #1045

New issue

Closed

opened 2026-02-27 09:10:42 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 09:10:42 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/970
Author: @nitnelave
Created: 9/7/2024
Status: ✅ Merged
Merged: 9/7/2024
Merged by: @nitnelave

Base: main ← Head: email

📝 Commits (1)

d448d49 server: clarify SMTP error message

📊 Changes

1 file changed (+8 additions, -4 deletions)

View changed files

📝 server/src/infra/mail.rs (+8 -4)

📄 Description

SMTP docs for many email providers use SSL to mean SSL/TLS, and TLS to mean STARTTLS, causing endless confusion. This should hopefully help.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/lldap/lldap/pull/970 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 9/7/2024 **Status:** ✅ Merged **Merged:** 9/7/2024 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `email` --- ### 📝 Commits (1) - [`d448d49`](https://github.com/lldap/lldap/commit/d448d49ff678ff06c8bd1629c0501d6384617696) server: clarify SMTP error message ### 📊 Changes **1 file changed** (+8 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `server/src/infra/mail.rs` (+8 -4) </details> ### 📄 Description SMTP docs for many email providers use SSL to mean SSL/TLS, and TLS to mean STARTTLS, causing endless confusion. This should hopefully help. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>