This is a tricky problem. I actually tried to fix this, but I didn't want to push a fix that was not complete.

Here is more to read about the regex approach in detecting emails: https://emailregex.com/

The relevant regex from the above website:

(?:[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)*|"(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21\x23-\x5b\x5d-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])*")@(?:(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?|\[(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?|[a-z0-9-]*[a-z0-9]:(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21-\x5a\x53-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])+)\])

I could just go ahead and use this, but I also get performance penalty. So I haven't yet found a valid solution for it.

My suggestion is to use the custom regex functionality of this lib and provide it the regex from the linked website above. The reason is simple that putting it as part of the library would affect everyone, while using the custom regex version is only limited to people who actually want it and agree to the performance penalty.

isEmail() doesnt work with XXX.YYY@ZZZ.com #6

Description

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions