Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

isMobilePhone with locale es-CU matches numbers of any length > 8 when the number starts with 5 #2196

Closed
klaframboise opened this issue Mar 3, 2023 · 0 comments · Fixed by #2197
Closed

isMobilePhone with locale es-CU matches numbers of any length > 8 when the number starts with 5 #2196

klaframboise opened this issue Mar 3, 2023 · 0 comments · Fixed by #2197
Labels
🐛 bug

Comments

@klaframboise
Copy link
Contributor

Describe the bug
isMobilePhone with locale es-CU matches any string that start with a 5 followed by at least 7 digits. This is because that locale's regex is missing the end of line matcher

Examples
Repl.it demo: https://replit.com/join/vcsakenqug-klaframboise1
Screenshot 2023-03-03 at 3 51 07 PM

Additional context
Validator.js version: 13.9.0
Node.js version: 18.12.1
OS platform: linux + macOS
@klaframboise klaframboise mentioned this issue Mar 3, 2023
Merged
4 tasks
profnandaa pushed a commit that referenced this issue Mar 7, 2023
@klaframboise
fix(isMobilePhone): fixed es-CU matching all numbers that start with …
cb91971 
…5 longer than 8 digits (#2197)

Co-authored-by: Kevin Laframboise <kevin@wisesystems.com>

fixes #2196
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛 bug
Projects
None yet
Milestone
No milestone
1 participant
@klaframboise