Bypass email validation for deleted emails

Author: Megamind51

src/asserts/email-assert.js

@@ -7,6 +7,17 @@
 const { Assert: is, Validator, Violation } = require('validator.js');
 let validator;
 
+/**
+ * Deleted email regex.
+ *
+ * The email should contain the following pattern:
+ * .+@.+\.any-number\.deleted
+ *
+ * example: [email protected]
+ */
+
+const deletedUserEmailRegex = /.+@.+\.\d+\.deleted$/;
+
 /**
  * Optional peer dependencies.
  */
@@ -41,16 +52,26 @@ module.exports = function emailAssert() {
       throw new Violation(this, value, { value: Validator.errorCode.must_be_a_string });
     }
 
-    if (!validator.isEmail(value)) {
-      throw new Violation(this, value);
-    }
-
     try {
       is.ofLength({ max: 254 }).validate(value);
     } catch (e) {
       throw new Violation(this, value);
     }
 
+    // We are bypassing the email validation for deleted users.
+    // This is needed because we have legacy users with invalid emails
+    // and as part of the deletion flow for expired signups we need to allow
+    // the deletion of that kind of users. The process of deleting a
+    // user updates their email from `${user.email}` to
+    // `${user.email}.${timestamp}.deleted`, so this assert runs on that update.
+    if (deletedUserEmailRegex.test(value) === true) {
+      return true;
+    }
+
+    if (!validator.isEmail(value)) {
+      throw new Violation(this, value);
+    }
+
     return true;
   };
 

test/asserts/email-assert.test.js

@@ -59,8 +59,29 @@ describe('EmailAssert', () => {
     }
   });
 
+  it('should throw an error on invalid emails', () => {
+    ['[email protected]', 'føø@båz.', 'foo+bar@baz_foo.com'].forEach(choice => {
+      try {
+        Assert.prototype.Email().validate(choice);
+
+        fail();
+      } catch (e) {
+        expect(e.show().assert).toBe('Email');
+      }
+    });
+  });
+
   it('should accept valid emails', () => {
-    ['[email protected]', 'føø@båz.com', '[email protected]'].forEach(choice => {
+    [
+      '[email protected]',
+      '[email protected]',
+      'føø@båz.com',
+      'føø@båz.com.123.deleted',
+      '[email protected]',
+      '[email protected]',
+      '[email protected]',
+      '[email protected]'
+    ].forEach(choice => {
       Assert.prototype.Email().validate(choice);
     });
   });