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

Fix: no auth method mailer #451

Closed
wants to merge 1 commit into from
Closed

Fix: no auth method mailer #451

wants to merge 1 commit into from

Conversation

jakerenzella
Copy link
Member

@jakerenzella jakerenzella commented Dec 17, 2024

WARNING: This change may require servers to set DF_SMTP_AUTHENTICATION to plain as that was the previous default.

Currently, mailer auth defaults to plain.

Some mailers don't rely on mailer auth, and instead use internal IP ranges.

No-auth method smtp relies on the lack of an auth key, and there is no valid value to default to.

The defaul plain method meant that non-auth mailers can't set the auth method correctly.

WARNING: This may require servers to set DF_SMTP_AUTHENTICATION to plain as that was the previous default.

@jakerenzella jakerenzella changed the base branch from development to 8.0.x December 17, 2024 04:52
@macite
Copy link
Member

macite commented Jan 29, 2025

Implemented through no_auth / none setting.

@macite macite closed this Jan 29, 2025
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