[sr-dev] git:master: kamailio.cfg: removed append_branch() from failure_route
Daniel-Constantin Mierla
miconda at gmail.com
Wed Jun 1 16:33:13 CEST 2011
Hello,
On 6/1/11 4:23 PM, Klaus Darilion wrote:
> Daniel, if I got it right, since Kamailio 3.1 such changes in failure
> route are automatically detected and append-branch is called
> automatically internally.
>
> As I couldn't find this documented:
>
> Which changes are detected? Any change to R-URI ($ru, $rU, sethostport
> ...)? Also changes to destination URI?
>
> Are there any other route blocks beside failure routes which trigger
> this behavior?
only the failure route had the issue in case of serial forking to create
a new branch.
I expect r-uri and dst-uri, but good question, since Andrei implemented
it I should look into the code.
Cheers,
Daniel
>
> Thanks
> Klaus
>
> Am 25.05.2011 22:55, schrieb Daniel-Constantin Mierla:
>> Module: sip-router
>> Branch: master
>> Commit: 9f542264b8d022959a2006af6fe73dd6b7cfbf13
>> URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=9f542264b8d022959a2006af6fe73dd6b7cfbf13
>>
>> Author: Daniel-Constantin Mierla<miconda at gmail.com>
>> Committer: Daniel-Constantin Mierla<miconda at gmail.com>
>> Date: Wed May 25 22:54:53 2011 +0200
>>
>> kamailio.cfg: removed append_branch() from failure_route
>>
>> - it is no longer necessary
>>
>> ---
>>
>> etc/kamailio.cfg | 4 +---
>> 1 files changed, 1 insertions(+), 3 deletions(-)
>>
>> diff --git a/etc/kamailio.cfg b/etc/kamailio.cfg
>> index 5e6d9f7..decfd0f 100644
>> --- a/etc/kamailio.cfg
>> +++ b/etc/kamailio.cfg
>> @@ -837,7 +837,7 @@ failure_route[FAIL_ONE] {
>> # uncomment the following lines if you want to block client
>> # redirect based on 3xx replies.
>> ##if (t_check_status("3[0-9][0-9]")) {
>> - ##t_reply("404","Not found");
>> + ## t_reply("404","Not found");
>> ## exit;
>> ##}
>>
>> @@ -845,8 +845,6 @@ failure_route[FAIL_ONE] {
>> # calls to a different new destination
>> ##if (t_check_status("486|408")) {
>> ## sethostport("192.168.2.100:5060");
>> - ## append_branch();
>> - ## # do not set the missed call flag again
>> ## t_relay();
>> ##}
>> }
>>
>>
>> _______________________________________________
>> sr-dev mailing list
>> sr-dev at lists.sip-router.org
>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
> _______________________________________________
> sr-dev mailing list
> sr-dev at lists.sip-router.org
> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
--
Daniel-Constantin Mierla
http://www.asipto.com
More information about the sr-dev
mailing list