Email validation fails with valid emails using new TLDs.

tgriesser / checkit

simple, flexible validations for node and the browser

MIT License

223 stars 53 forks source link

Email validation fails with valid emails using new TLDs. #30

Closed rhys-vdw closed 9 years ago

rhys-vdw commented 9 years ago

This email fails validation: x@example.melbourne.

/^[a-z0-9._%+\-]+@[a-z0-9.\-]+\.[a-z]{2,6}$/i

A limit of 6 characters is not sufficient. According to this Stack Overflow answer the maximum length is 63 characters, not 6. I will provide a PR today to address this.

rhys-vdw commented 9 years ago

Hi @tgriesser. Is this project still active? Been waiting a while for a response on this PR. I'm currently using Checkit (sparingly) in production and would like some confirmation if possible. Cheers.