Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update settings for Strato #114

Merged
merged 1 commit into from
May 10, 2024
Merged

Update settings for Strato #114

merged 1 commit into from
May 10, 2024

Conversation

cketti
Copy link
Member

@cketti cketti commented May 8, 2024

They removed support for CRAM-MD5 and don't support any other encrypted authentication methods.

See https://www.strato.de/faq/mail/externes-e-mail-programm-mit-strato-e-mail-adresse-nutzen/

(I am a customer and was notified of this via email)

@cketti cketti requested a review from babolivier May 8, 2024 18:05
Copy link
Member

@babolivier babolivier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can see a few opportunities for drive-by improvements to this config:

@cketti
Copy link
Member Author

cketti commented May 10, 2024

I added the link to the documentation to the config.

There are two SMTP entries. The first one uses TLS (port 465), the second one STARTTLS (port 587). A couple of other configs also contain multiple entries for the same protocol. The general idea is that clients that don't support the first one hopefully support the second one. So I'd like to leave it like this for now.

However, I don't think any modern email client can get away with only supporting one of TLS or STARTTLS. So multiple entries are redundant.
I'll try to find some time to write up a proposal for a policy document on what we want to include in a provider config.

Copy link
Member

@babolivier babolivier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, somehow I missed we already had the non-STARTTLS entry, my bad.

@cketti cketti merged commit 04c2c13 into master May 10, 2024
1 check passed
@cketti cketti deleted the update_strato branch May 10, 2024 14:07
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.

2 participants