Closed jamesholcomb closed 2 years ago
Any chance you could submit a PR to fix with a test?
On Sunday, November 14, 2021, James Holcomb @.***> wrote:
const emailRegEx = require('email-regex-safe');const validator = require('validator'); @.')); // trueconsole.log(emailRegEx().test('email me @.')); // falseconsole.log(emailRegEx().test('email me @.***')); // true
Thanks for putting together this repo.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/niftylettuce/email-regex-safe/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD7XBNAVSYB6TWYUW7RQLDUL7H47ANCNFSM5H7634NA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
How did you come up with the local-part regex?
@jamesholcomb this actually isn't a bug.
You need to add gmail: false
to options for emailRegexSafe
. The local part is not a valid gmail local part but otherwise it is.
Thanks for putting together this repo.