Hello,
based on the discussion from last weeks on sr-dev related to selection of reply code for failure route, I did some updates in kamailio_3.0 branch. Main goal was to have the option of Kamailio 1.5 compatibility mode. Here is the patch:
http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3ed66850...
Description of the changes is included in the commit message.
Andrei, can you double check if there is any possible side effect considering the new TM architecture inherited from ser? Patch is not that complex.
Miklos, does it meet what we tried to cover in the discussion? Anything missing?
Very much appreciated if anyone can test and send feedback.
Cheers, Daniel
On Dec 02, 2009 at 19:17, Daniel-Constantin Mierla miconda@gmail.com wrote:
Hello,
based on the discussion from last weeks on sr-dev related to selection of reply code for failure route, I did some updates in kamailio_3.0 branch. Main goal was to have the option of Kamailio 1.5 compatibility mode. Here is the patch:
http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3ed66850...
Description of the changes is included in the commit message.
Andrei, can you double check if there is any possible side effect considering the new TM architecture inherited from ser? Patch is not that complex.
It looks ok to me. The only minor problem is that the test for TM_UAC_FLAG_FB should be done only #ifdef TM_UAC_FLAGS (if TM_UAC_FLAGS is not defined one won't be able to do t_drop_replies("last") because the "first" branch is no longer marked).
Andrei
Hello Daniel,
On Wed, 2 Dec 2009, Daniel-Constantin Mierla wrote:
Hello,
based on the discussion from last weeks on sr-dev related to selection of reply code for failure route, I did some updates in kamailio_3.0 branch. Main goal was to have the option of Kamailio 1.5 compatibility mode. Here is the patch:
http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3ed66850...
Description of the changes is included in the commit message.
Andrei, can you double check if there is any possible side effect considering the new TM architecture inherited from ser? Patch is not that complex.
Miklos, does it meet what we tried to cover in the discussion? Anything missing?
yes, this is all that we discussed. Thanks!
Miklos
Very much appreciated if anyone can test and send feedback.
Cheers, Daniel
-- Daniel-Constantin Mierla
-
Andrei Pelinescu-Onciul -
Daniel-Constantin Mierla -
Miklos Tirpak