Skip to content
New issue

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

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

Already on GitHub? # to your account

Non breaking space bypass email validation #2218

Closed
Gaspar-Cdv opened this issue Apr 14, 2023 · 1 comment · Fixed by #2237
Closed

Non breaking space bypass email validation #2218

Gaspar-Cdv opened this issue Apr 14, 2023 · 1 comment · Fixed by #2237

Comments

@Gaspar-Cdv
Copy link

"test test@test.com" won't pass since it's a regular space (0x0020), but "test test@test.com" will pass since it's a non breaking space (0x00A0).

In fact, /\s/ regex pattern is not checking for non breaking spaces.

Here is a thread about it: https://stackoverflow.com/questions/31790440/regex-to-replace-no-break-space

@jeremy21212121
Copy link
Contributor

I've created a PR (#2237) that hopefully addresses this issue in a satisfactory manner. I'd appreciate some feedback if a maintainer has the chance.

profnandaa pushed a commit that referenced this issue Jun 26, 2023
Co-authored-by: jpoole <jpoole@maxmind.com>

fixes #2218
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants