<p>ims_qos: create a proper flow description for the SIP signaling</p>

<ul>
<li>before there was just an empty one in the AAR</li>
<li>re-using the function typically used for the RTP media</li>
<li>fixing then that function to not just hard-code protocol 17 (UDP) for everything</li>
<li>now supporting also 6 (TCP) and IP (IP has no protocol number, we use just the
magic word "ip" (see RFC 3588 IPFilterRule for more details)</li>
<li>adding the IP of the P-CSCF as a parameter to the module, as this is required
in the flow (UE IP <-> P-CSCF IP)</li>
<li>also some inline/static fixes to function for avoiding warnings on gcc >=5</li>
</ul>

<p>ims_registrar_pcscf: fixed removal of contacts on de-registration</p>

<ul>
<li>the code was there, but I guess some API changes resulted in temporary
commented-out code; this uses the new parameters and seems to fix the
issue.</li>
</ul>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/kamailio/kamailio/pull/721'>https://github.com/kamailio/kamailio/pull/721</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>ims_qos: create a proper flow description for the SIP signaling</li>
  <li>ims_registrar_pcscf: fixed removal of contacts on de-registration</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/721/files#diff-0">modules/ims_qos/mod.c</a>
    (4)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/721/files#diff-1">modules/ims_qos/rx_aar.c</a>
    (32)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/721/files#diff-2">modules/ims_qos/rx_avp.c</a>
    (47)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/721/files#diff-3">modules/ims_qos/rx_avp.h</a>
    (12)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/721/files#diff-4">modules/ims_registrar_pcscf/save.c</a>
    (6)
  </li>
</ul>

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

<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/721">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZaOSYV_bXgWot_1gq0LWG6kRj-ePks5qXlOVgaJpZM4JQ_SP">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZclNliE5AKrF5Hrab4a5iMPRnehPks5qXlOVgaJpZM4JQ_SP.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/pull/721"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>