[sr-dev] append_branch() change

[Daniel-Constantin Mierla]

  Lack of time here prevented me to test as well. Anyhow, this remembers 
me that I have to remove related code from dispatcher, since makes no 
sense to keep it there anymore.

Cheers,
Daniel


On 9/30/10 12:19 PM, Andrei Pelinescu-Onciul wrote:
> On Sep 30, 2010 at 12:05, Miklos Tirpak<miklos at iptel.org>  wrote:
>> hi,
>>
>> I was recently fighting with a serial forking issue, and figured out
>> that t_relay() automatically appends a new branch when the RURI has
>> been changed. This means to me that the config below becomes
>> incorrect:
>>
>> failure_route["abcd"] {
>> 		...
>>                  rewritehostport("127.0.0.1:5062");
>>                  append_branch();
>>                  t_relay();
>> }
>>
>> I saw two branches created with the same RURI and sent to the same
>> destination in parallel.
>>
>> As far as I got it append_branch() should not be called any more in
>> failure route unless parallel forking is desired. This is great, the
>> only issue is that this is not mentioned in the documentation, the
>> tm readme file still contains the old examples, and all the
>> configurations under etc/ still call append_branch(). Is there any
>> plan to update the docs and configs before releasing the new
>> sip-router version?
>
> There is a plan to try to avoid this situation by making append_branch()
>   not have any effect in the above case.
>
> I even have a patch (attached)  for it (Daniel discovered the same
> problem as you), but so far I did not have any time to test it.
>
> Nevertheless we should update the docs, migration document and the
> configs.
>
> Andrei
>
>
> _______________________________________________
> 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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20100930/c156c091/attachment.htm>