[kamailio/kamailio] tm: check branch uri against current uri when deciding if append a ne… (#702)
8 Jul
2016
8 Jul
'16
3:57 a.m.
…w branch
This patch allows t_append_branches to cope with sip.instance usage, at least for the case
when the contact uri has changed with regards to a previous registration.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/702
-- Commit Summary --
* tm: check branch uri against current uri when deciding if append a new branch
-- File Changes --
M modules/tm/t_append_branches.c (4)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/702.patch
https://github.com/kamailio/kamailio/pull/702.diff
---
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/702
8 Jul
8 Jul
4:15 a.m.
You can merge it.
I think it covers the most of the cases, but some corner ones may not be covered. Like one
behind a nat router that is changing the public ip -- so the contact is the same, but the
received address (computed from source ip) will be different. I guess this can happen only
if the phone did a re-register at the time was called and the router changed its up more
or less at that moment.
---
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/702#issuecomment-231301616
4:35 a.m.
New subject: [sr-dev] [kamailio/kamailio] tm: check branch uri against current uri when deciding if append a ne… (#702)
You're right. Maybe adding also the check on the received address? I didn't
want to have a too heavy logic there, but it could make sense.
On Fri, Jul 8, 2016 at 10:15 AM, Daniel-Constantin Mierla <
notifications(a)github.com> wrote:
You can merge it.
I think it covers the most of the cases, but some corner ones may not be
covered. Like one behind a nat router that is changing the public ip -- so
the contact is the same, but the received address (computed from source ip)
will be different. I guess this can happen only if the phone did a
re-register at the time was called and the router changed its up more or
less at that moment.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<https://github.com/kamailio/kamailio/pull/702#issuecomment-231301616>,
or mute the thread
<https://github.com/notifications/unsubscribe/AF36ZT0QvLLfY34XElcOpcg4tOgq6gsLks5qTgcJgaJpZM4JHzZD>
.
_______________________________________________
sr-dev mailing list
sr-dev(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
2915
days inactive
2915
days old
2 comments
3 participants
participants (3)
-
Daniel-Constantin Mierla -
Federico Cabiddu
-
Federico Cabiddu