<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://reviewboard.asterisk.org/r/1216/">https://reviewboard.asterisk.org/r/1216/</a>
     </td>
    </tr>
   </table>
   <br />





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This fixes the instance where we are matching the extension in get_destination, but what about every other place sip URI&#39;s are parsed.  For instance, isn&#39;t this a problem when matching peer names as well?</pre>
 <br />







<p>- David</p>


<br />
<p>On May 13th, 2011, 10:05 a.m., jrose wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Asterisk Developers, Russell Bryant, David Vossel, and Leif Madsen.</div>
<div>By jrose.</div>


<p style="color: grey;"><i>Updated 2011-05-13 10:05:32</i></p>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This is the latest in a long series of attempts to address an interoperability concern mentioned in the bug.  SIP URIs are including semicolon delimited user parameters and trying to get rid of them without something overkill like eating strings starting at semicolons is tough.  This option is primarily for people who are having this particular problem and aren&#39;t wise to the cool stuff you can do with pattern matching and dialplan functions and control statements.  That way when they pop up in chat saying their extension wasn&#39;t found when it was RIGHT THERE, we can quickly tell them &quot;Yeah, you can enable this option right here.  Or well, you could go read this book *plug book link here* and learn to do it the awesome way.&quot;  And nine times out of ten they&#39;ll probably be satisfied with this easier method since putting semicolons into extensions requires some level of voodoo involving setting dialplan in DB or at the very least, using an escape character (and using backslash in front of the semicolon also includes the backslash in the extension for some reason, which is weird.  Without backslash, it just comments out the rest and no extension is made.  This might be a bug.).

So anyway, here&#39;s an option to strip everything after a semicolon for extension matching.  It&#39;s basically the same as the very first patch I made for this, but it seems like that&#39;s what is wanted at the moment.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">sipp to send a request for the extension with and without the option enabled.

actual user field:
2005;phone-context=+1;npdi=yes

matches extension without option:
2005;phone-context=+1;npdi=yes

matches extension with option:
2005</pre>
  </td>
 </tr>
</table>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="https://issues.asterisk.org/view.php?id=18344">18344</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>/branches/1.8/channels/chan_sip.c <span style="color: grey">(318634)</span></li>

 <li>/branches/1.8/channels/sip/include/sip.h <span style="color: grey">(318634)</span></li>

 <li>/branches/1.8/configs/sip.conf.sample <span style="color: grey">(318634)</span></li>

</ul>

<p><a href="https://reviewboard.asterisk.org/r/1216/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>