This patch helps us to handle inadequatness of Linphone mobile apps (both iOS and Android flavours are affected) without reinventing a lot of logics to handle each transport (we use TCP, TLS, WS, WSS).
E.g. if we have two TLS-based Linphone clients, even with enable_double_rr for gentle cross-forwarding we would get single RR header without transport= attribute, resulting in Linphone sending ACK and BYE via UDP.
This new option, being enabled, results in two RR headers being always inserted, with transport attribute being always added. I believe this cannot harm the installation with enable_double_rr, just makes instructions more explicit.
The Linphone's behaviour looks like a set of different bugs, but it would get us much more efforts to fix.
I don't know if any other useragent needs such workaround. So I would perfectly understand if this commit wouldn't be accepted upstream.

Also this commit currently lacks README doc rebuild, because I don't know how to do that. Only XML doc is updated.

You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/257

Commit Summary

• rr: add enable_double_rr_always option

File Changes

• M modules/rr/doc/rr_admin.xml (24)
• M modules/rr/record.c (14)
• M modules/rr/rr_mod.c (2)
• M modules/rr/rr_mod.h (1)
• M tcp_read.c (5)

Patch Links:

• https://github.com/kamailio/kamailio/pull/257.patch
• https://github.com/kamailio/kamailio/pull/257.diff

—
Reply to this email directly or view it on GitHub.