10000 Fix eMail address regex by mirabilos · Pull Request #14719 · angular/angular.js · GitHub
[go: up one dir, main page]
Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Fix eMail address regex #14719

Closed
wants to merge 7 commits into from
Closed

Fix eMail address regex #14719

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
ae87315
Fix eMail address local-part validation
mirabilos Jun 6, 2016
baf0fa7
Limit eMail address individual host part length
mirabilos Jun 6, 2016
215a60f
Limit eMail address total host part length
mirabilos Jun 6, 2016
28292fd
Optimise eMail address regex for speed
mirabilos Jun 6, 2016
f3e0116
expand ranges in regex character class to improve legibility
mirabilos Jun 6, 2016
8720cff
Comprehensive eMail addr-spec testsuite
mirabilos Jun 6, 2016
c410413
Limit size of local-part and total path size in eMail addresses
mirabilos Jun 6, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Comprehensive eMail addr-spec testsuite
  • Loading branch information
@mirabilos
mirabilos committed Jun 6, 2016
commit 8720cff5ddc513697c2a95dbfd18cadba5ca18e9
86 changes: 86 additions & 0 deletions test/ng/directive/inputSpec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2784,14 +2784,100 @@ describe('input', function() {
describe('EMAIL_REGEXP', function() {
/* global EMAIL_REGEXP: false */
it('should validate email', function() {
/* basic functionality */
expect(EMAIL_REGEXP.test('a@b.com')).toBe(true);
expect(EMAIL_REGEXP.test('a@b.museum')).toBe(true);
expect(EMAIL_REGEXP.test('a@B.c')).toBe(true);
/* domain label separation, hyphen-minus, syntax */
expect(EMAIL_REGEXP.test('a@b.c.')).toBe(false);
expect(EMAIL_REGEXP.test('a@.b.c')).toBe(false);
expect(EMAIL_REGEXP.test('a@-b.c')).toBe(false);
expect(EMAIL_REGEXP.test('a@b-.c')).toBe(false);
expect(EMAIL_REGEXP.test('a@b-c')).toBe(true);
expect(EMAIL_REGEXP.test('a@-')).toBe(false);
expect(EMAIL_REGEXP.test('a@.')).toBe(false);
expect(EMAIL_REGEXP.test('a@host_name')).toBe(false);
/* leading or sole digit */
expect(EMAIL_REGEXP.test('a@3b.c')).toBe(true);
expect(EMAIL_REGEXP.test('a@3')).toBe(true);
/* TLD eMail address */
expect(EMAIL_REGEXP.test('a@b')).toBe(true);
/* domain valid characters */
expect(EMAIL_REGEXP.test('a@abcdefghijklmnopqrstuvwxyz-ABCDEFGHIJKLMNOPQRSTUVWXYZ.0123456789')).toBe(true);
/* domain invalid characters */
expect(EMAIL_REGEXP.test('a@')).toBe(false);
expect(EMAIL_REGEXP.test('a@ ')).toBe(false);
expect(EMAIL_REGEXP.test('a@!')).toBe(false);
expect(EMAIL_REGEXP.test('a@"')).toBe(false);
expect(EMAIL_REGEXP.test('a@#')).toBe(false);
expect(EMAIL_REGEXP.test('a@$')).toBe(false);
expect(EMAIL_REGEXP.test('a@%')).toBe(false);
expect(EMAIL_REGEXP.test('a@&')).toBe(false);
expect(EMAIL_REGEXP.test("a@'")).toBe(false);
expect(EMAIL_REGEXP.test('a@(')).toBe(false);
expect(EMAIL_REGEXP.test('a@)')).toBe(false);
expect(EMAIL_REGEXP.test('a@*')).toBe(false);
expect(EMAIL_REGEXP.test('a@+')).toBe(false);
expect(EMAIL_REGEXP.test('a@,')).toBe(false);
expect(EMAIL_REGEXP.test('a@/')).toBe(false);
expect(EMAIL_REGEXP.test('a@:')).toBe(false);
expect(EMAIL_REGEXP.test('a@;')).toBe(false);
expect(EMAIL_REGEXP.test('a@<')).toBe(false);
expect(EMAIL_REGEXP.test('a@=')).toBe(false);
expect(EMAIL_REGEXP.test('a@>')).toBe(false);
expect(EMAIL_REGEXP.test('a@?')).toBe(false);
expect(EMAIL_REGEXP.test('a@@')).toBe(false);
expect(EMAIL_REGEXP.test('a@[')).toBe(false);
expect(EMAIL_REGEXP.test('a@\\')).toBe(false);
expect(EMAIL_REGEXP.test('a@]')).toBe(false);
expect(EMAIL_REGEXP.test('a@^')).toBe(false);
expect(EMAIL_REGEXP.test('a@_')).toBe(false);
expect(EMAIL_REGEXP.test('a@`')).toBe(false);
expect(EMAIL_REGEXP.test('a@{')).toBe(false);
expect(EMAIL_REGEXP.test('a@|')).toBe(false);
expect(EMAIL_REGEXP.test('a@}')).toBe(false);
expect(EMAIL_REGEXP.test('a@~')).toBe(false);
expect(EMAIL_REGEXP.test('a@İ')).toBe(false);
expect(EMAIL_REGEXP.test('a@ı')).toBe(false);
/* domain length, label and total */
expect(EMAIL_REGEXP.test('a@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx')).toBe(true);
expect(EMAIL_REGEXP.test('a@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx')).toBe(false);
/* jshint maxlen:320 */
expect(EMAIL_REGEXP.test('a@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx')).toBe(true);
expect(EMAIL_REGEXP.test('a@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.x')).toBe(true);
expect(EMAIL_REGEXP.test('a@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.xx')).toBe(false);
/* jshint maxlen:200 */
/* local-part valid characters and dot-atom syntax */
expect(EMAIL_REGEXP.test("'@x")).toBe(true);
expect(EMAIL_REGEXP.test('-!#$%&*+/0123456789=?ABCDEFGHIJKLMNOPQRSTUVWXYZ^_`abcdefghijklmnopqrstuvwxyz{|}~@x')).toBe(true);
/* jshint maxlen:320 */
expect(EMAIL_REGEXP.test('-!#$%&*+/0123456789=?ABCDEFGHIJKLMNOPQRSTUVWXYZ^_`abcdefghijklmnopqrstuvwxyz{|}~.-!#$%&*+/0123456789=?ABCDEFGHIJKLMNOPQRSTUVWXYZ^_`abcdefghijklmnopqrstuvwxyz{|}~.-!#$%&*+/0123456789=?ABCDEFGHIJKLMNOPQRSTUVWXYZ^_`abcdefghijklmnopqrstuvwxyz{|}~@x')).toBe(true);
/* jshint maxlen:200 */
expect(EMAIL_REGEXP.test(".@x")).toBe(false);
expect(EMAIL_REGEXP.test("'.@x")).toBe(false);
expect(EMAIL_REGEXP.test(".'@x")).toBe(false);
expect(EMAIL_REGEXP.test("'.'@x")).toBe(true);
/* local-part invalid characters */
expect(EMAIL_REGEXP.test('@x')).toBe(false);
expect(EMAIL_REGEXP.test(' @x')).toBe(false);
expect(EMAIL_REGEXP.test('"@x')).toBe(false);
expect(EMAIL_REGEXP.test('(@x')).toBe(false);
expect(EMAIL_REGEXP.test(')@x')).toBe(false);
expect(EMAIL_REGEXP.test(',@x')).toBe(false);
expect(EMAIL_REGEXP.test(':@x')).toBe(false);
expect(EMAIL_REGEXP.test(';@x')).toBe(false);
expect(EMAIL_REGEXP.test('<@x')).toBe(false);
expect(EMAIL_REGEXP.test('>@x')).toBe(false);
expect(EMAIL_REGEXP.test('@@x')).toBe(false);
expect(EMAIL_REGEXP.test('[@x')).toBe(false);
expect(EMAIL_REGEXP.test('\\@x')).toBe(false);
expect(EMAIL_REGEXP.test(']@x')).toBe(false);
expect(EMAIL_REGEXP.test('İ@x')).toBe(false);
expect(EMAIL_REGEXP.test('ı@x')).toBe(false);
/* content (local-part + ‘@’ + domain) is required */
expect(EMAIL_REGEXP.test('')).toBe(false);
expect(EMAIL_REGEXP.test('a')).toBe(false);
expect(EMAIL_REGEXP.test('aa')).toBe(false);
});
});
});
Expand Down
0