fix(isIBAN): only strip spaces and hyphens before validating IBAN (#1268

)
validatorjs · Mar 15, 2020 · 04769b5 · 04769b5
1 parent afafb54
commit 04769b5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/lib/isIBAN.js b/src/lib/isIBAN.js
@@ -94,12 +94,12 @@ const ibanRegexThroughCountryCode = {
  * @return {boolean}
  */
 function hasValidIbanFormat(str) {
-  // Strip white spaces and hyphens, keep only digits and A-Z latin alphabetic
-  const strippedStr = str.replace(/[^A-Z0-9]+/gi, '').toUpperCase();
+  // Strip white spaces and hyphens
+  const strippedStr = str.replace(/[\s\-]+/gi, '').toUpperCase();
   const isoCountryCode = strippedStr.slice(0, 2).toUpperCase();
 
   return (isoCountryCode in ibanRegexThroughCountryCode) &&
-      ibanRegexThroughCountryCode[isoCountryCode].test(strippedStr);
+    ibanRegexThroughCountryCode[isoCountryCode].test(strippedStr);
 }
 
 /**

diff --git a/test/validators.js b/test/validators.js
@@ -3825,6 +3825,9 @@ describe('Validators', () => {
       invalid: [
         'XX22YYY1234567890123',
         'FR14 2004 1010 0505 0001 3',
+        'FR7630006000011234567890189@',
+        'FR7630006000011234567890189😅',
+        'FR763000600001123456!!🤨7890189@',
       ],
     });
   });