<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=5906877" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/gaaf">@gaaf</a> - I see that dlg_update_rr_set() is used only for DLG_CALLER_LEG, no need to do it for CALLEE as well?</p>
<p>Having an option to update Record-Route sets within dialog can be useful, I think at this moment they are stored only during the call setup.</p>
<p>On the other hand, for what you try to achieve, I wonder if it would not be simpler to use somehow the caller/callee sockets fields, they are already stored inside the dialog record and they should overlap in address, port and transport with local record-route. So based on modparam, the local requests are sent to the caller/callee socket, with top record route built at storage time or at send time (store vs cpu question here, like none that relevant to decide for one or the other option) -- this just as an idea, I haven't really analyzed if it is a simpler solution, I just thought that those addresses are already stored.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/pull/1832#issuecomment-462518197">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZQqs11FlmKo5VmR2-vqhc7_2NMdFks5vMeyigaJpZM4aeP-h">mute the thread</a>.<img src="https://github.com/notifications/beacon/AF36ZQmrEqPKA_Blr2WHIuIttvFYddzFks5vMeyigaJpZM4aeP-h.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kamailio/kamailio","title":"kamailio/kamailio","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"PERSON","message":"@miconda in #1832: @gaaf - I see that dlg_update_rr_set() is used only for DLG_CALLER_LEG, no need to do it for CALLEE as well?\r\n\r\nHaving an option to update Record-Route sets within dialog can be useful, I think at this moment they are stored only during the call setup.\r\n\r\nOn the other hand, for what you try to achieve, I wonder if it would not be simpler to use somehow the caller/callee sockets fields, they are already stored inside the dialog record and they should overlap in address, port and transport with local record-route. So based on modparam, the local requests are sent to the caller/callee socket, with top record route built at storage time or at send time (store vs cpu question here, like none that relevant to decide for one or the other option) -- this just as an idea, I haven't really analyzed if it is a simpler solution, I just thought that those addresses are already stored. "}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1832#issuecomment-462518197"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/kamailio/kamailio/pull/1832#issuecomment-462518197",
"url": "https://github.com/kamailio/kamailio/pull/1832#issuecomment-462518197",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>