-
Notifications
You must be signed in to change notification settings - Fork 79
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
Not valid email #43
Comments
yeah, it doesn't check the TLDs from iana |
@manishsaraan If someone submitted a pull request for domain validation (disabled by default, enabled by an optional function parameter), would you accept it? It would include the IANA list. Could be helpful for some users, even though it's separate from the RFC. I'd be interested in working on it if it would match the goals of the project. |
Thanks for suggestion @wpapper . Validating domains is not main goal of this project but i agree with your suggestion to add options to enable it. I will happily merge pr for this feature. |
If you do this, please do make it optional. Remember, there are potentially private domains that are perfectly valid. |
|
|
validator.validate("test@email.comdddddd"); // return true
The text was updated successfully, but these errors were encountered: