<p>Hi <a href="https://github.com/miconda" class="user-mention">@miconda</a> I just read through some of the history on this issue and see you have been adding commits around this issue from before June 25th. The other issue I saw <a href="https://github.com/kamailio/kamailio/issues/1165" class="issue-link js-issue-link" data-url="https://github.com/kamailio/kamailio/issues/1165" data-id="238333477" data-error-text="Failed to load issue title" data-permission-text="Issue title is private">#1165</a> appears to be the same problem. Couldn't we get around this issue by simply adding ":5060" to the after the ip when the th_uri_prefix is generated? If it is added with the port into the SIP message then I feel like it would be safe to assume the carriers will not try to modify the URI and screw up the prefix thus allowing you to keep your URI prefix safety checks by default. Any thoughts?</p>
<p>I believe the changes you made to 4.4 should fix my issue. I created a small temporary patch myself and I will be testing shortly by just appending ":5060" to the uri prefix to see if I can validate my thoughts.</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/issues/1222#issuecomment-326025513">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZSZGaPUA94G8Ecujjz5IBENpxuAAks5sdX5agaJpZM4PGoWl">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZcTC4pvkTpVk3QA-MPKHWyAVKdelks5sdX5agaJpZM4PGoWl.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/kamailio/kamailio/issues/1222#issuecomment-326025513"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>

<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://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"PERSON","message":"@rampageai in #1222: Hi @miconda I just read through some of the history on this issue and see you have been adding commits around this issue from before June 25th. The other issue I saw #1165 appears to be the same problem. Couldn't we get around this issue by simply adding \":5060\" to the after the ip when the th_uri_prefix is generated? If it is added with the port into the SIP message then I feel like it would be safe to assume the carriers will not try to modify the URI and screw up the prefix thus allowing you to keep your URI prefix safety checks by default. Any thoughts?\r\n\r\nI believe the changes you made to 4.4 should fix my issue. I created a small temporary patch myself and I will be testing shortly by just appending \":5060\" to the uri prefix to see if I can validate my thoughts."}],"action":{"name":"View Issue","url":"https://github.com/kamailio/kamailio/issues/1222#issuecomment-326025513"}}}</script>