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.

Sign up for GitHub

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

Jump to bottom

Emails with multiple @ symbols pass validation #86

Closed
narwold opened this issue Aug 3, 2022 · 1 comment
Closed

Emails with multiple @ symbols pass validation #86

narwold opened this issue Aug 3, 2022 · 1 comment

Comments

@narwold
Copy link

narwold commented Aug 3, 2022

We had a test in one of our codebases that expected asdf@asdfasd@asdfsdf.com to fail validation, and our test is now failing with the 1.7.0 upgrade because that email now seems to be considered valid.
alexdunae added a commit that referenced this issue Aug 3, 2022
@alexdunae
Fix allowed symbols in atom-text
9d66b38 
'@' symbols and other invalid characters were being allowed due to a faulty regexp

Fixes #86
@narwold
Copy link
Author

narwold commented Aug 10, 2022

Thanks @alexdunae!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@narwold