Sorry for the late feedback. I did try it with some of our suspend scenarios
and I haven't noticed side effects. No load though.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/971#i
Merged #971.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/971#event-961348396___
sr-dev mailing list
sr-dev@lists.sip-router.org
http://
Can be merged and backported to 5.0, eventually later to 4.4.
Probably the deep cloning from acc makes no longer sense, I will check that
myself, for now should be harmless.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
h
I did test with our scenarios, I didn't detect any side effect
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/971#issuecomment-279691834__
@linuxmaniac, @grumvalski - wondering if you had any time running some tests
with the patch, just to have a feeling if you observed any side effect.
I didn't get to review yet, planning to do it during the next days.
--
You are receiving this because you are subscribed to this thread.
Reply to
seems ok to me.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/971#issuecomment-278269542___
sr-dev mailing list
sr-dev@lists.sip-router.o
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/971
-- Commit Summary --
* tm: deep cloning of the request for fake environment
-- File Changes --
M src/modules/tm/t_append_branches.c (23)
M src/modules/tm/t_reply.c (104)