Skip to content

[12.x] Make it more clear what is happening with email RFC validation #10340

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

Merged
merged 1 commit into from
Apr 22, 2025

Conversation

tanerkay
Copy link
Contributor

As of a commit since 12.x, the email validation docs could be seen as misleading/confusing.

The email validation does not validate against RFC 5322, but also allows other RFCs. This is important, as other RFCs allow emails that are not compatible with several (most?) major email platforms.

Also, the strict validation mentions "rejecting trailing periods or multiple consecutive periods", which I think is an oversimplification.

I don't see a way in the upstream egulias/EmailValidator to selectively validate against certain RFCs.

As the list of RFCs is subject to change, I propose changing the docs to mention "supported RFCs", with a link to the list. The link might be deemed superfluous as the package itself is linked to earlier in this docs section.

Alternative approach would be to delegate to the egulias/EmailValidator docs for the rfc and strict rule styles.

Related topics:

@tanerkay tanerkay changed the title Make it more clear what is happening with email RFC validation [12.x] Make it more clear what is happening with email RFC validation Apr 22, 2025
@taylorotwell taylorotwell merged commit 0ca1cfe into laravel:12.x Apr 22, 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