30 Oct
2017
30 Oct
'17
10:33 a.m.
Hello,
On 27.10.17 16:23, Juha Heinanen wrote:
I noticed that K 5.0 sanity() test passes uri that
contains ` (back quote)
character.
Config:
modparam("sanity", "default_checks", 1024) /* URI checks */
modparam("sanity", "uri_checks", 3) /* RURI, From */
xlog("L_INFO", "Checking $ru\n");
if (!sanity_check())
xlog("L_INFO", "Check failed\n");
else
xlog("L_INFO", "Check passed\n");
Syslog:
Oct 27 17:17:38 lohi /usr/bin/sip-proxy[31946]: INFO: Checking sip:jh@te`st.fi
Oct 27 17:17:38 lohi /usr/bin/sip-proxy[31946]: INFO: Check passed
According to RFC3261:
hostname = *( domainlabel "." ) toplabel [ "." ]
domainlabel = alphanum
/ alphanum *( alphanum / "-" ) alphanum
Is this a bug or am I missing something?
I don't remember any change recently to sanity, so probably it was
developed like this for long time. You can go ahead and fix it in the
code. As a quick fix, you can add an extra regexp condition in the config.
Cheers,
Daniel
--
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio Advanced Training, Nov 13-15, 2017, in Berlin - www.asipto.com
Kamailio World Conference - www.kamailioworld.com