fix: `@IsPhoneNumber` decorator does not match E.164 format

[ihorTymofieiev]

Description

Hi, I use the class-validator library to validate request data on a Node.js app. I have problems with validating phone numbers with intl. prefix (use 'ZZ' as a region)!

Example

class PhoneNumberWithIssue {
  @IsNotEmpty({ message: 'ERROR_EMPTY_PHONE_NUMBER_FIELD' })
  @IsPhoneNumber('ZZ', { message: 'ERROR_INVALID_PHONE_NUMBER' })
  phoneNumber!: string;
}

const invalidPhoneNumber = new PhoneNumberWithIssue();
invalidPhoneNumber.phoneNumber = "+380981111111++++++++++=====++++=+";
validate(invalidPhoneNumber).then(errors => {
    if (errors.length > 0) {
        console.log("validation failed. errors: ", errors);
    } else {
        console.log("validation succeed"); // Validation is succeeded, but the phone number is not valid!
    }
});

Dependencies

• "class-validator": "^0.10.0"

[vlapo]

Looks like this is the way how to libphonenumber library parse strings. From javadocs:

Parses a string and returns it as a phone number in proto buffer format. The method is quite lenient and looks for a number in the input text (raw input) and does not check whether the string is definitely only a phone number. To do this, it ignores punctuation and white-space, as well as any text before the number (e.g. a leading "Tel: ") and trims the non-number bits. It will accept a number in any format (E164, national, international etc), assuming it can be interpreted with the defaultRegion supplied. It also attempts to convert any alpha characters into digits if it thinks this is a vanity number of the type "1800 MICROSOFT".

So the question is. Is there any possible option how to do more strict parse? I dont think so #421 is good way how to handle this problem E164 is not only valid format.

[ihorTymofieiev]

Hmm, then #421 is not a solution. I will try to make a more universal validation.

[vlapo]

There are plenty of ports https://github.com/google/libphonenumber library to another programming languages. I hope someone have a similar issue. Maybe check official issue tracker for https://github.com/google/libphonenumber first. Some solution could be there. :) I will try check this too.

[ihorTymofieiev]

https://github.com/google/libphonenumber ignores all characters after the phone number. For them, this is normal. You can find notes about this in their documentation and as a comment in the source code.

[rubiin]

@vlapo can we just use something like this https://github.com/validatorjs/validator.js/blob/master/lib/isMobilePhone.js

[NoNameProvided]

The related PR has been closed as out-of-date because the libary used to parse phone numbers has changed. This issue needs to be re-tested with the new lib.