From 222caa16be91db326bc7bff441963b8c0a032a72 Mon Sep 17 00:00:00 2001 From: Sarhan Date: Thu, 4 Mar 2021 23:54:14 +0100 Subject: [PATCH 1/2] fix(isMobilePhone): prevent validator from allowing landline numbers in es-CO locale --- src/lib/isMobilePhone.js | 2 +- test/validators.js | 14 +++++++------- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/lib/isMobilePhone.js b/src/lib/isMobilePhone.js index 68a19922e..8f81d2df3 100644 --- a/src/lib/isMobilePhone.js +++ b/src/lib/isMobilePhone.js @@ -56,7 +56,7 @@ const phones = { 'en-ZW': /^(\+263)[0-9]{9}$/, 'es-AR': /^\+?549(11|[2368]\d)\d{8}$/, 'es-BO': /^(\+?591)?(6|7)\d{7}$/, - 'es-CO': /^(\+?57)?([1-8]{1}|3[0-9]{2})?[0-9]{1}\d{6}$/, + 'es-CO': /^(\+?57|03)?3(0|1|2|5)\d{8}$/, 'es-CL': /^(\+?56|0)[2-9]\d{1}\d{7}$/, 'es-CR': /^(\+506)?[2-8]\d{7}$/, 'es-DO': /^(\+?1)?8[024]9\d{7}$/, diff --git a/test/validators.js b/test/validators.js index 34b6474e8..23b3968f6 100644 --- a/test/validators.js +++ b/test/validators.js @@ -6506,18 +6506,11 @@ describe('Validators', () => { valid: [ '+573003321235', '573003321235', - '579871235', '3003321235', '3213321235', '3103321235', '3253321235', - '3321235', - '574321235', - '5784321235', - '5784321235', - '9821235', '573011140876', - '0698345', ], invalid: [ '1234', @@ -6530,6 +6523,13 @@ describe('Validators', () => { '5703013347567', '069834567', '969834567', + '579871235', + '574321235', + '5784321235', + '5784321235', + '9821235', + '0698345', + '3321235', ], }, { From 9ab2f2baa9525eb662d2a44a4c4999cdf5df872b Mon Sep 17 00:00:00 2001 From: Sarhan Date: Fri, 5 Mar 2021 14:47:41 +0100 Subject: [PATCH 2/2] fix(isMobilePhone): prevent es-CO validator from allowing invalid number --- src/lib/isMobilePhone.js | 2 +- test/validators.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/lib/isMobilePhone.js b/src/lib/isMobilePhone.js index 8f81d2df3..207dae71b 100644 --- a/src/lib/isMobilePhone.js +++ b/src/lib/isMobilePhone.js @@ -56,7 +56,7 @@ const phones = { 'en-ZW': /^(\+263)[0-9]{9}$/, 'es-AR': /^\+?549(11|[2368]\d)\d{8}$/, 'es-BO': /^(\+?591)?(6|7)\d{7}$/, - 'es-CO': /^(\+?57|03)?3(0|1|2|5)\d{8}$/, + 'es-CO': /^(\+?57)?3(0(0|1|2|4|5)|1\d|2[0-4]|5(0|1))\d{7}$/, 'es-CL': /^(\+?56|0)[2-9]\d{1}\d{7}$/, 'es-CR': /^(\+506)?[2-8]\d{7}$/, 'es-DO': /^(\+?1)?8[024]9\d{7}$/, diff --git a/test/validators.js b/test/validators.js index 23b3968f6..d6d02c540 100644 --- a/test/validators.js +++ b/test/validators.js @@ -6509,7 +6509,7 @@ describe('Validators', () => { '3003321235', '3213321235', '3103321235', - '3253321235', + '3243321235', '573011140876', ], invalid: [