[WW-3885] Invalid email address validation (client side). EmailValidator regex implementation doesn't match documentation.

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.3.4.1
• Fix Version/s: 2.3.7
• Component/s: Core Interceptors, Documentation
• Labels: patch
• Flags: Important
• Assignee: Lukasz Lenart
• Reporter: Martin Uhlir
• Created: 07/Oct/12 11:16
• Updated: 05/Nov/12 19:45
• Resolved: 11/Oct/12 10:01

Description

EmailValidator documentation specifies following regular expression to validate email addresses:

\\b(^[_A-Za-z0-9-]+(\\.[_A-Za-z0-9-]+)*@([A-Za-z0-9-])+((\\.com)|(\\.net)|(\\.org)|(\\.info)|(\\.edu)|(\\.mil)|(\\.gov)|(\\.biz)|(\\.ws)|(\\.us)|(\\.tv)|(\\.cc)|(\\.aero)|(\\.arpa)|(\\.coop)|(\\.int)|(\\.jobs)|(\\.museum)|(\\.name)|(\\.pro)|(\\.travel)|(\\.nato)|(\\..{2,3})|(\\..{2,3}\\..{2,3}))$)\\b

but the actual implementation uses following regular expression:

\\b(^['_A-Za-z0-9-]+(\\.['_A-Za-z0-9-]+)*@([A-Za-z0-9-])+(\\.[A-Za-z0-9-]+)*((\\.[A-Za-z0-9]{2,})|(\\.[A-Za-z0-9]{2,}\\.[A-Za-z0-9]{2,}))$)\\b

which is wrong, because this email address "[e-mail]" is being validated as a correct email address (I experience this issue in client side validation).

The (more?) correct regular expression should be probably the one specified in the documentation, but care needs to be taken of the apostrophe for the front part of the email address as well. The resulting regular expression might be something like this:

\\b(^['_A-Za-z0-9-]+(\\.['_A-Za-z0-9-]+)*@([A-Za-z0-9-])+((\\.com)|(\\.net)|(\\.org)|(\\.info)|(\\.edu)|(\\.mil)|(\\.gov)|(\\.biz)|(\\.ws)|(\\.us)|(\\.tv)|(\\.cc)|(\\.aero)|(\\.arpa)|(\\.coop)|(\\.int)|(\\.jobs)|(\\.museum)|(\\.name)|(\\.pro)|(\\.travel)|(\\.nato)|(\\..{2,3})|(\\..{2,3}\\..{2,3}))$)\\b

Issue Links

• relates to: WW-3883 EmailValidator does not allow to use + (plus sign) in emails