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: email with digits is a valid phone #1002

Closed
antelle opened this issue Mar 13, 2019 · 2 comments
Closed

isMobilePhone: email with digits is a valid phone #1002

antelle opened this issue Mar 13, 2019 · 2 comments
Labels
🔍needs-investigation

Comments

@antelle
Copy link

antelle commented Mar 13, 2019

Hi!

There are some emails that pass the phone validation, for example:

isMobilePhone('[email protected]')

returns true.

It happens because of this regex: 'bn-BD': /\+?(88)?0?1[356789][0-9]{8}\b/,.

Probably it should have ^ like other phones, but I'm not sure about the format.
@profnandaa
Copy link
Member

Will check and revert, but a PR is also welcome.

@alouini333 alouini333 mentioned this issue May 17, 2019
Merged
@chriso
Copy link
Collaborator

chriso commented May 23, 2019

Fixed by #1032.

@chriso chriso closed this as completed May 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🔍needs-investigation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@profnandaa @chriso @antelle