With topoh enabled I have:
Route: sip:10.56.42.37:5070;lr;ftag=IpXq-LwNvJF6WrkOcmZNf9WjmGjK5dUv;did=90c.1132;vsf=AAAAAAAAAAAAAAAAAABlY2Bmf359HGJ6cX8bc3h9OjUwNjA7dXNlcj1waG9uZQ--;vst=AAAAAAAAAAAAAAAAAAAAAAAAAAAAAgEAAgcAAAABAAAAAAAAAAAAAAAA Route: sip:TEST.MSS.LIFE.COM:5060;transport=UDP;lr
But with topos enabled I have:
Route: sip:TEST.MSS.LIFE.COM:5060;transport=UDP;lr,sip:10.56.42.37:5070;lr;ftag=EN0cvoXXCOgdGoO2zizq-WNmE4Enf4Nt;did=434.5ce1;vsf=AAAAAAAAAAAAAAAAAABlY2Bmf359HGJ6cX8bc3h9OjUwNjA7dXNlcj1waG9uZQ--;vst=AAAAAAAAAAAAAAAAAAAAAAAAAAAAAgEAAgcAAAABAAAAAAAAAAAAAAAA
And kamailio trying to send ACK to TEST.MSS.LIFE.COM:5060
After this patch Route header are restores in from:
Route: sip:10.56.42.37:5070;lr;ftag=EN0cvoXXCOgdGoO2zizq-WNmE4Enf4Nt;did=434.5ce1;vsf=AAAAAAAAAAAAAAAAAABlY2Bmf359HGJ6cX8bc3h9OjUwNjA7dXNlcj1waG9uZQ--;vst=AAAAAAAAAAAAAAAAAAAAAAAAAAAAAgEAAgcAAAABAAAAAAAAAAAAAAAA Route: sip:TEST.MSS.LIFE.COM:5060;transport=UDP;lr
Now ACK are routed correctly. You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1124
-- Commit Summary --
* topos: correct order of Route header after restoring from b_rr
-- File Changes --
M src/modules/topos/tps_msg.c (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1124.patch https://github.com/kamailio/kamailio/pull/1124.diff
Merged manually to simplify the commit log message. Extensive descrition with logs and traces should be done only in PR, not in commit message, there should be a summary of what it fixes.
Closed #1124.