30 Sep
2010
30 Sep
'10
12:05 p.m.
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?
Thanks,
Miklos
30 Sep
30 Sep
12:19 p.m.
On Sep 30, 2010 at 12:05, Miklos Tirpak <miklos(a)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
12:35 p.m.
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(a)iptel.org> wrote:
--
Daniel-Constantin Mierla
http://www.asipto.com
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(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
12:47 p.m.
Hi Andrei,
I just tested the patch both with and without calling append_branch(),
and it worked. I also tried two additional scenarios which were a bit
more complicated, both of them worked as expected:
a) serial forking with 3 branches:
1. request route: t_relay()
2. failure_route1: rewrite URI, append_branch(), t_realy()
3. failure_route2: rewrite URI, append_branch(), t_realy()
b) serial forking + parallel forking from failure route
1. request route: t_relay()
2. failure_route: rewrite URI, append_branch(),
rewrite URI, append_branch(),
t_relay()
Thanks,
Miklos
On 09/30/2010 12:19 PM, Andrei Pelinescu-Onciul wrote:
On Sep 30, 2010 at 12:05, Miklos
Tirpak<miklos(a)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
1:21 p.m.
On Sep 30, 2010 at 12:47, Miklos Tirpak <miklos(a)iptel.org> wrote:
Hi Andrei,
I just tested the patch both with and without calling
append_branch(), and it worked. I also tried two additional
scenarios which were a bit more complicated, both of them worked as
expected:
a) serial forking with 3 branches:
1. request route: t_relay()
2. failure_route1: rewrite URI, append_branch(), t_realy()
3. failure_route2: rewrite URI, append_branch(), t_realy()
b) serial forking + parallel forking from failure route
1. request route: t_relay()
2. failure_route: rewrite URI, append_branch(),
rewrite URI, append_branch(),
t_relay()
Thanks, commited.
Andrei
Thanks,
Miklos
On 09/30/2010 12:19 PM, Andrei Pelinescu-Onciul wrote:
On Sep 30, 2010 at 12:05, Miklos
Tirpak<miklos(a)iptel.org> wrote:
_______________________________________________
sr-dev mailing list
sr-dev(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev 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
4969
days inactive
4969
days old
4 comments
3 participants
participants (3)
-
Andrei Pelinescu-Onciul -
Daniel-Constantin Mierla -
Miklos Tirpak