Put length restrictions on email parts, re #258

test/validators.js

@@ -32,6 +32,14 @@ function test(options) {
     }
 }
 
+function repeat(str, count) {
+    var result = '';
+    while (count--) {
+        result += str;
+    }
+    return result;
+}
+
 describe('Validators', function () {
 
     it('should validate email addresses', function () {
@@ -51,6 +59,7 @@ describe('Validators', function () {
               , '"foobar"@example.com'
               , '"  foo  m端ller "@example.com'
               , '"foo\\@bar"@example.com'
+              , repeat('a', 64) + '@' + repeat('a', 252) + '.com'
             ]
           , invalid: [
                 'invalidemail@'
@@ -60,6 +69,9 @@ describe('Validators', function () {
               , 'somename@ｇｍａｉｌ.com'
               , '[email protected].'
               , '[email protected]'
+              , 'ｇｍａｉｌｇｍａｉｌｇｍａｉｌｇｍａｉｌｇｍａｉｌ@gmail.com'
+              , repeat('a', 64) + '@' + repeat('a', 253) + '.com'
+              , repeat('a', 65) + '@' + repeat('a', 252) + '.com'
             ]
         });
     });

validator.js

@@ -187,6 +187,11 @@
             user = user.replace(/\./g, '').toLowerCase();
         }
 
+        if (!validator.isByteLength(user, 0, 64) ||
+                !validator.isByteLength(domain, 0, 256)) {
+            return false;
+        }
+
         if (!validator.isFQDN(domain, {require_tld: options.require_tld})) {
             return false;
         }