No subject


Thu Mar 26 14:29:42 CET 2015


Without this patch, OpenSIPS will only reuse connections targetted at the IP / ports in the Via, which correspond to the actual currently active connection. But most of the time, the proxy will not use those IP/ports when communicating with the UA. It will instead use the registered IP/ports. We should automatically reuse the active connection for all communications with those IP/ports instead of creating new connections, especially in 1.8 where it is a blocking operation.

If the connection can not be reused, then OpenSIPS will create a new one.
You can view, comment on, or merge this pull request online at:

  https://github.com/OpenSIPS/opensips/pull/446

-- Commit Summary --

  * Registrar: Add contact-based aliasing when the remote UA supports it.

-- File Changes --

    M modules/registrar/save.c (13)

-- Patch Links --

https://github.com/OpenSIPS/opensips/pull/446.patch
https://github.com/OpenSIPS/opensips/pull/446.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/pull/446

----==_mimepart_551a6e3c7a2d6_5c863fa8f8f752c027399b
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

<p>If an endpoint supporting TCP connection reuse registers to OpenSIPS, we<br>
now create TCP connections aliases for all IP/ports announced by the UA.</p>

<p>From my point of view, it is more a fix than a new feature.</p>

<p>Without this patch, OpenSIPS will only reuse connections targetted at the IP / ports in the Via, which correspond to the actual currently active connection. But most of the time, the proxy will not use those IP/ports when communicating with the UA. It will instead use the registered IP/ports. We should automatically reuse the active connection for all communications with those IP/ports instead of creating new connections, especially in 1.8 where it is a blocking operation.</p>

<p>If the connection can not be reused, then OpenSIPS will create a new one.</p>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>&nbsp;&nbsp;<a href='https://github.com/OpenSIPS/opensips/pull/446'>https://github.com/OpenSIPS/opensips/pull/446</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>Registrar: Add contact-based aliasing when the remote UA supports it.</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/OpenSIPS/opensips/pull/446/files#diff-0">modules/registrar/save.c</a>
    (13)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/OpenSIPS/opensips/pull/446.patch'>https://github.com/OpenSIPS/opensips/pull/446.patch</a></li>
  <li><a href='https://github.com/OpenSIPS/opensips/pull/446.diff'>https://github.com/OpenSIPS/opensips/pull/446.diff</a></li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href="https://github.com/OpenSIPS/opensips/pull/446">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AFOcierWRgNBkn08JZk8HB00EFqoNjr3ks5n6mW8gaJpZM4D3pwO.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/OpenSIPS/opensips/pull/446"></link>
    <meta itemprop="name" content="View Pull Request"></meta>
  </div>
  <meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>


----==_mimepart_551a6e3c7a2d6_5c863fa8f8f752c027399b--



More information about the Devel mailing list