i have this piece of config code:
if (is_gruu()) { xlog("L_INFO", "Request-URI <$ru> IS gruu\n"); } else { xlog("L_INFO", "Request-URI <$ru> is NOT gruu\n"); }
and i get this to syslog:
May 28 17:53:07 siika /usr/sbin/sip-proxy[8926]: INFO: Request-URI sip:jh@test.fi;gr=urn:uuid:f462ab09-88fe-43c6-9c9c-cb570ce6bb99 is NOT gruu
is this a bug or have i misunderstood something?
-- juha
4.24. is_gruu([uri])
The function returns true if the uri is GRUU ('gr' parameter is present): 1 - pub-gruu; 2 - temp-gruu.
Meaning of the parameters is as follows:
uri - the SIP URI to check for GRUU parameter. It is optional, when missing, the value of R-URI is used.
Juha Heinanen writes:
i have this piece of config code:
if (is_gruu()) { xlog("L_INFO", "Request-URI <$ru> IS gruu\n"); } else { xlog("L_INFO", "Request-URI <$ru> is NOT gruu\n");}
and i get this to syslog:
May 28 17:53:07 siika /usr/sbin/sip-proxy[8926]: INFO: Request-URI sip:jh@test.fi;gr=urn:uuid:f462ab09-88fe-43c6-9c9c-cb570ce6bb99 is NOT gruu
is this a bug or have i misunderstood something?
the above test succeeded when i changed it to:
if (is_gruu($ru)) {
perhaps is_gruu() does not work when $ru has been changed in the script?
-- juha
Hello,
I just pushed a patch that should fix it. There was a wrong return code check for r-uri parsing function. Let me know if still not working.
Cheers, Daniel
On 5/28/12 7:06 PM, Juha Heinanen wrote:
Juha Heinanen writes:
i have this piece of config code:
if (is_gruu()) { xlog("L_INFO", "Request-URI <$ru> IS gruu\n"); } else { xlog("L_INFO", "Request-URI <$ru> is NOT gruu\n");}
and i get this to syslog:
May 28 17:53:07 siika /usr/sbin/sip-proxy[8926]: INFO: Request-URI sip:jh@test.fi;gr=urn:uuid:f462ab09-88fe-43c6-9c9c-cb570ce6bb99 is NOT gruu
is this a bug or have i misunderstood something?
the above test succeeded when i changed it to:
if (is_gruu($ru)) {
perhaps is_gruu() does not work when $ru has been changed in the script?
-- juha
sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
-
Daniel-Constantin Mierla -
Juha Heinanen