How about this?
https://github.com/kamailio/kamailio/pull/2643
-- Juha
___
Kamailio (SER) - Users Mailing List
sr-users@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
Looks like loose.c is_myself() function only uses host/port of the first
Route URI to determine if Route URI denotes the proxy itself thus
ignoring ;sn parameter.
-- Juha
___
Kamailio (SER) - Users Mailing List
sr-users@lists.kamailio.org
https://lists.
Just to add that loose_route() works OK if I add to config
alias=tenantX.teams.tutpro.com:8005
So that sort of proves that ;ns=ext_tls param in the first Route URI has
no effect.
-- Juha
---
> I'm returning to this thread, since after some mo
I'm returning to this thread, since after some more testing, I found that
loose_route() does not (at least always) detect that a pair of route
headers denotes itself.
In config I have defined:
listen=tls:192.x.x.x:8005 name "ext_tls"
modparam("rr", "sockname_mode", 1)
and Route header looks lik
Daniel-Constantin Mierla writes:
> > Also, there is no "=". So instead of ";sn=", the code should look for
> > "sn".
>
> Indeed, I will push a fix for it.
I also did one, but yours is cleaner than mine. Thanks. Tested and it
works, no more warnings to syslog.
Looks like no-one had ever before
Juha Heinanen writes:
> I think I found the bug:
>
> ./src/core/config.h:#define SOCKNAME_PARAM ";sn="
>
> but in parse_params() result params are without ";".
Also, there is no "=". So instead of ";sn=", the code should look for
"sn".
-- Juha
___
I think I found the bug:
./src/core/config.h:#define SOCKNAME_PARAM ";sn="
but in parse_params() result params are without ";".
-- Juha
___
Kamailio (SER) - Users Mailing List
sr-users@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/list
Daniel-Constantin Mierla writes:
> Do you have the rr modparam related socket name mode set?
Yes, sorry, forgot to mention it:
modparam("rr", "sockname_mode", 1)
I have added some debug to loose.c and some converted DBGs to INFOs in
rr_do_force_send_socket() function:
if(rr_sockname_mo
On 12.02.21 15:44, Juha Heinanen wrote:
> Daniel-Constantin Mierla writes:
>
>>> Also, there is no "=". So instead of ";sn=", the code should look for
>>> "sn".
>> Indeed, I will push a fix for it.
> I also did one, but yours is cleaner than mine. Thanks. Tested and it
> works, no more warnings
On 12.02.21 15:24, Daniel-Constantin Mierla wrote:
> On 12.02.21 15:20, Juha Heinanen wrote:
>> Juha Heinanen writes:
>>
>>> I think I found the bug:
>>>
>>> ./src/core/config.h:#define SOCKNAME_PARAM ";sn="
>>>
>>> but in parse_params() result params are without ";".
>> Also, there is no "=". S
On 12.02.21 15:20, Juha Heinanen wrote:
> Juha Heinanen writes:
>
>> I think I found the bug:
>>
>> ./src/core/config.h:#define SOCKNAME_PARAM ";sn="
>>
>> but in parse_params() result params are without ";".
> Also, there is no "=". So instead of ";sn=", the code should look for
> "sn".
Indeed
I did more tests and found that loose_route() does not find socket by
name given in ;sn param:
Feb 12 10:40:59 lab /usr/bin/sip-proxy[16975]: WARNING: rr [loose.c:799]:
rr_do_force_send_socket(): no socket found to match second RR
(sip:tenantX.teams.tutpro.com:8007;transport=tls;r2=on;sn=ext_tls
Do you have the rr modparam related socket name mode set?
Cheers,
Daniel
On 12.02.21 09:55, Juha Heinanen wrote:
> I did more tests and found that loose_route() does not find socket by
> name given in ;sn param:
>
> Feb 12 10:40:59 lab /usr/bin/sip-proxy[16975]: WARNING: rr [loose.c:799]:
> rr_d
13 matches
Mail list logo
