Hello list, Hope you are all doing fine!
I am using the is_e164() function to validate the number we receive, and I come to see that the number +555 was accepted.... After some googling it looks like(it is not very clear though) that 7 digits are the minimum we could have for e164 numbers but after checking the source code, I saw it accepts anything starting with + and having between 2 and 16 numbers. So is it really valid to have a number with just 2 digits? What is the case?
Thank you, Kind regards, Patrick Wakano
Hello, it's strange that why this function (actually i mean e164_check function) doesn’t check the length of number! Based on Recommendation ITU-T E.164, Annex A.3 Number length, it should be matched with one of structures that is explained there So you could do check it yourself with regex. With Regards. Mojtaba On Mon, Dec 3, 2018 at 8:39 AM Patrick Wakano pwakano@gmail.com wrote:
Hello list, Hope you are all doing fine!
I am using the is_e164() function to validate the number we receive, and I come to see that the number +555 was accepted.... After some googling it looks like(it is not very clear though) that 7 digits are the minimum we could have for e164 numbers but after checking the source code, I saw it accepts anything starting with + and having between 2 and 16 numbers. So is it really valid to have a number with just 2 digits? What is the case?
Thank you, Kind regards, Patrick Wakano _______________________________________________ Kamailio (SER) - Users Mailing List sr-users@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
Am Montag, 3. Dezember 2018, 06:08:27 CET schrieb Patrick Wakano:
I am using the is_e164() function to validate the number we receive, and I come to see that the number +555 was accepted.... After some googling it looks like(it is not very clear though) that 7 digits are the minimum we could have for e164 numbers but after checking the source code, I saw it accepts anything starting with + and having between 2 and 16 numbers. So is it really valid to have a number with just 2 digits? What is the case?
Hello Patrick,
I think the implementation was done with a pragmatic approach, to make sure that we don't reject numbers that are used in the field. The ITU standard Amendment A mentions the possibility to use national short numbers, for example. The standard mentions that the maximal length should be 15, but I think in this case this was also implemented a bit more relaxed.
The original implementation from the enum module allows even longer numbers, I will check if this should be synchronized.
Best regards,
Henning
Thanks for the replies guys! I will probably add a length test to invalidate too short numbers! By the way, I had a quick look in the ITU recommendation ( https://www.itu.int/rec/T-REC-E.164/en) and looks like the short numbers are for local purposes only being part of the "Non-ITU-T E.164 numbers" section, so I guess the function should return false in this case. Also it seems the + sign is recommended but not mandatory for a E.164 number, which is quite confusing.....
Kind regards, Patrick Wakano
On Tue, 4 Dec 2018 at 08:57, Henning Westerholt hw@kamailio.org wrote:
Am Montag, 3. Dezember 2018, 06:08:27 CET schrieb Patrick Wakano:
I am using the is_e164() function to validate the number we receive, and
I
come to see that the number +555 was accepted.... After some googling it looks like(it is not very clear though) that 7 digits are the minimum we could have for e164 numbers but after checking the source code, I saw it accepts anything starting with + and having between 2 and 16 numbers. So is it really valid to have a number with
just
2 digits? What is the case?
Hello Patrick,
I think the implementation was done with a pragmatic approach, to make sure that we don't reject numbers that are used in the field. The ITU standard Amendment A mentions the possibility to use national short numbers, for example. The standard mentions that the maximal length should be 15, but I think in this case this was also implemented a bit more relaxed.
The original implementation from the enum module allows even longer numbers, I will check if this should be synchronized.
Best regards,
Henning
-- Henning Westerholt - https://skalatan.de/blog/ Kamailio services - https://skalatan.de/services Kamailio security assessment - https://skalatan.de/de/assessment
You might also want to have a look at the “phonenum” module...
https://www.kamailio.org/docs/modules/5.2.x/modules/phonenum.html
On Tue, Dec 4, 2018 at 19:12 Patrick Wakano pwakano@gmail.com wrote:
Thanks for the replies guys! I will probably add a length test to invalidate too short numbers! By the way, I had a quick look in the ITU recommendation ( https://www.itu.int/rec/T-REC-E.164/en) and looks like the short numbers are for local purposes only being part of the "Non-ITU-T E.164 numbers" section, so I guess the function should return false in this case. Also it seems the + sign is recommended but not mandatory for a E.164 number, which is quite confusing.....
Kind regards, Patrick Wakano
On Tue, 4 Dec 2018 at 08:57, Henning Westerholt hw@kamailio.org wrote:
Am Montag, 3. Dezember 2018, 06:08:27 CET schrieb Patrick Wakano:
I am using the is_e164() function to validate the number we receive,
and I
come to see that the number +555 was accepted.... After some googling it looks like(it is not very clear though) that 7 digits are the minimum we could have for e164 numbers but after checking the source code, I saw it accepts anything starting with + and having between 2 and 16 numbers. So is it really valid to have a number with
just
2 digits? What is the case?
Hello Patrick,
I think the implementation was done with a pragmatic approach, to make sure that we don't reject numbers that are used in the field. The ITU standard Amendment A mentions the possibility to use national short numbers, for example. The standard mentions that the maximal length should be 15, but I think in this case this was also implemented a bit more relaxed.
The original implementation from the enum module allows even longer numbers, I will check if this should be synchronized.
Best regards,
Henning
-- Henning Westerholt - https://skalatan.de/blog/ Kamailio services - https://skalatan.de/services Kamailio security assessment - https://skalatan.de/de/assessment
Kamailio (SER) - Users Mailing List sr-users@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
Thanks for the hint Joel! That's a really nice addition! One more good reason to upgrade to version 5.2!!
Cheers, Patrick Wakano
On Wed, 5 Dec 2018 at 14:30, Joel Serrano joel@textplus.com wrote:
You might also want to have a look at the “phonenum” module...
https://www.kamailio.org/docs/modules/5.2.x/modules/phonenum.html
On Tue, Dec 4, 2018 at 19:12 Patrick Wakano pwakano@gmail.com wrote:
Thanks for the replies guys! I will probably add a length test to invalidate too short numbers! By the way, I had a quick look in the ITU recommendation ( https://www.itu.int/rec/T-REC-E.164/en) and looks like the short numbers are for local purposes only being part of the "Non-ITU-T E.164 numbers" section, so I guess the function should return false in this case. Also it seems the + sign is recommended but not mandatory for a E.164 number, which is quite confusing.....
Kind regards, Patrick Wakano
On Tue, 4 Dec 2018 at 08:57, Henning Westerholt hw@kamailio.org wrote:
Am Montag, 3. Dezember 2018, 06:08:27 CET schrieb Patrick Wakano:
I am using the is_e164() function to validate the number we receive,
and I
come to see that the number +555 was accepted.... After some googling it looks like(it is not very clear though) that 7 digits are the minimum we could have for e164 numbers but after
checking
the source code, I saw it accepts anything starting with + and having between 2 and 16 numbers. So is it really valid to have a number with
just
2 digits? What is the case?
Hello Patrick,
I think the implementation was done with a pragmatic approach, to make sure that we don't reject numbers that are used in the field. The ITU standard Amendment A mentions the possibility to use national short numbers, for example. The standard mentions that the maximal length should be 15, but I think in this case this was also implemented a bit more relaxed.
The original implementation from the enum module allows even longer numbers, I will check if this should be synchronized.
Best regards,
Henning
-- Henning Westerholt - https://skalatan.de/blog/ Kamailio services - https://skalatan.de/services Kamailio security assessment - https://skalatan.de/de/assessment
Kamailio (SER) - Users Mailing List sr-users@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
Kamailio (SER) - Users Mailing List sr-users@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
I think the function was mainly made to check if it is a telephone number in international format.
It could be better to make this function more strict in following the e.164 specs, there is another function to check if it looks like a telephone number, and as pointed in another response, there is a module for more strict checking on validity of a phone number per country...
Cheers, Daniel
On 05.12.18 04:10, Patrick Wakano wrote:
Thanks for the replies guys! I will probably add a length test to invalidate too short numbers! By the way, I had a quick look in the ITU recommendation (https://www.itu.int/rec/T-REC-E.164/en) and looks like the short numbers are for local purposes only being part of the "Non-ITU-T E.164 numbers" section, so I guess the function should return false in this case. Also it seems the + sign is recommended but not mandatory for a E.164 number, which is quite confusing.....
Kind regards, Patrick Wakano
On Tue, 4 Dec 2018 at 08:57, Henning Westerholt <hw@kamailio.org mailto:hw@kamailio.org> wrote:
Am Montag, 3. Dezember 2018, 06:08:27 CET schrieb Patrick Wakano: > I am using the is_e164() function to validate the number we receive, and I > come to see that the number +555 was accepted.... > After some googling it looks like(it is not very clear though) that 7 > digits are the minimum we could have for e164 numbers but after checking > the source code, I saw it accepts anything starting with + and having > between 2 and 16 numbers. So is it really valid to have a number with just > 2 digits? What is the case? Hello Patrick, I think the implementation was done with a pragmatic approach, to make sure that we don't reject numbers that are used in the field. The ITU standard Amendment A mentions the possibility to use national short numbers, for example. The standard mentions that the maximal length should be 15, but I think in this case this was also implemented a bit more relaxed. The original implementation from the enum module allows even longer numbers, I will check if this should be synchronized. Best regards, Henning -- Henning Westerholt - https://skalatan.de/blog/ Kamailio services - https://skalatan.de/services Kamailio security assessment - https://skalatan.de/de/assessment
Kamailio (SER) - Users Mailing List sr-users@lists.kamailio.org https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
Thanks for your reply Daniel! I agree with you, currently this function name is somewhat misleading...
Cheers, Patrick Wakano
On Mon, 10 Dec 2018 at 18:31, Daniel-Constantin Mierla miconda@gmail.com wrote:
I think the function was mainly made to check if it is a telephone number in international format.
It could be better to make this function more strict in following the e.164 specs, there is another function to check if it looks like a telephone number, and as pointed in another response, there is a module for more strict checking on validity of a phone number per country...
Cheers, Daniel On 05.12.18 04:10, Patrick Wakano wrote:
Thanks for the replies guys! I will probably add a length test to invalidate too short numbers! By the way, I had a quick look in the ITU recommendation ( https://www.itu.int/rec/T-REC-E.164/en) and looks like the short numbers are for local purposes only being part of the "Non-ITU-T E.164 numbers" section, so I guess the function should return false in this case. Also it seems the + sign is recommended but not mandatory for a E.164 number, which is quite confusing.....
Kind regards, Patrick Wakano
On Tue, 4 Dec 2018 at 08:57, Henning Westerholt hw@kamailio.org wrote:
Am Montag, 3. Dezember 2018, 06:08:27 CET schrieb Patrick Wakano:
I am using the is_e164() function to validate the number we receive,
and I
come to see that the number +555 was accepted.... After some googling it looks like(it is not very clear though) that 7 digits are the minimum we could have for e164 numbers but after checking the source code, I saw it accepts anything starting with + and having between 2 and 16 numbers. So is it really valid to have a number with
just
2 digits? What is the case?
Hello Patrick,
I think the implementation was done with a pragmatic approach, to make sure that we don't reject numbers that are used in the field. The ITU standard Amendment A mentions the possibility to use national short numbers, for example. The standard mentions that the maximal length should be 15, but I think in this case this was also implemented a bit more relaxed.
The original implementation from the enum module allows even longer numbers, I will check if this should be synchronized.
Best regards,
Henning
-- Henning Westerholt - https://skalatan.de/blog/ Kamailio services - https://skalatan.de/services Kamailio security assessment - https://skalatan.de/de/assessment
Kamailio (SER) - Users Mailing Listsr-users@lists.kamailio.orghttps://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
-- Daniel-Constantin Mierla -- www.asipto.comwww.twitter.com/miconda -- www.linkedin.com/in/miconda Kamailio World Conference -- www.kamailioworld.com Kamailio Advanced Training, Nov 12-14, 2018, in Berlin -- www.asipto.com
-
Daniel-Constantin Mierla -
Henning Westerholt -
Joel Serrano -
Mojtaba -
Patrick Wakano