<p>I split the pull request in three commits to seperate the fix from the new feature, but they go together so I propose them in the same pull request:</p>

<p><a href="https://github.com/kamailio/kamailio/commit/cc0b07d2bcadbd95b2f0dbfdcc873306acca8f2d" class="commit-link"><tt>cc0b07d</tt></a>: I added some log message that would have helped me to understand a 400 Bad Request response.<br>
<a href="https://github.com/kamailio/kamailio/commit/c0697d76fc2ae8ea9676aa9a9b57db442b9c678f" class="commit-link"><tt>c0697d7</tt></a>: I fixed a bug that were the contact size is not checked on the first header field and 200 OK is sent instead of 400 Bad Request for too long contact URIs but the uri ends up beein truncated by the database.<br>
<a href="https://github.com/kamailio/kamailio/commit/eb6b9c6bbd3674dc9e2b6da80b010986dd300e3f" class="commit-link"><tt>eb6b9c6</tt></a>: I made the contact size configurable to allow longer contacts</p>

<hr>

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

<h4>Commit Summary</h4>
<ul>
  <li>registrar: Add warnings if contact is invalid and REGISTER will be rejected</li>
  <li>registrar: Check max URI size of contact also for first contact header</li>
  <li>registrar: Add module parameter "contact_max_size" to make max contact size configurable</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/725/files#diff-0">modules/registrar/doc/registrar_admin.xml</a>
    (30)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/725/files#diff-1">modules/registrar/registrar.c</a>
    (3)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/725/files#diff-2">modules/registrar/registrar.h</a>
    (2)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/725/files#diff-3">modules/registrar/sip_msg.c</a>
    (19)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/kamailio/kamailio/pull/725.patch'>https://github.com/kamailio/kamailio/pull/725.patch</a></li>
  <li><a href='https://github.com/kamailio/kamailio/pull/725.diff'>https://github.com/kamailio/kamailio/pull/725.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/725">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZVJ9Q0C0FcPf9cL48Bnasigvbf8lks5qZct9gaJpZM4JU7Nt">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZRFYTg-rmll0zjCHKIxpQPrfgee1ks5qZct9gaJpZM4JU7Nt.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/725"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>