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 Tirpakmiklos@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@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev