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(Mailer): Allow to enforce strict email format #45570

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

susnux
Copy link
Contributor

@susnux susnux commented May 29, 2024

Alternative to #45425

But this one instead set the global email verification to strict.

@susnux susnux force-pushed the fix/strict-email-verification branch from ad0dc36 to 250991a Compare May 31, 2024 10:01
@susnux susnux requested a review from come-nc May 31, 2024 10:01
@susnux susnux force-pushed the fix/strict-email-verification branch from 250991a to 4f64c35 Compare May 31, 2024 11:05
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the fix/strict-email-verification branch from 4f64c35 to 1a27314 Compare May 31, 2024 12:18
@susnux susnux merged commit c1661b6 into master Jun 4, 2024
155 checks passed
@susnux susnux deleted the fix/strict-email-verification branch June 4, 2024 12:03
@susnux
Copy link
Contributor Author

susnux commented Jun 4, 2024

/backport to stable29

@joshtrichards
Copy link
Member

joshtrichards commented Jul 11, 2024

I think this is currently breaking CI in master (at least over at suspicious_login, not sure why it isn't here in server currently):

https://github.com/nextcloud/suspicious_login/actions/runs/9779149403/job/26997666619?pr=887
https://github.com/nextcloud/suspicious_login/actions/runs/9894385660/job/27331763334?pr=889

Or some related BC introduced maybe via nextcloud/3rdparty#1811

@blizzz blizzz mentioned this pull request Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants