<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 />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 13th, 2011, 10:12 a.m., <b>David Vossel</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<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's are parsed. For instance, isn't this a problem when matching peer names as well?</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I imagine it very easily could, but the only scenario the bug reporter mentioned was the case where he was trying to make a call and couldn't connect to the extension. So really, I don't know much about how it matches peer names at the moment. I also don't know if there is any need to fix it up to match peer names. I don't think the reporter or any of the responders mentioned anything about problems relating to them, and for all I know at the moment, the device in question might not be throwing in all these user parameters for things like REGISTER, SUBSCRIBE, or whatever else uses a peer name.
So yeah... hard to say if it's a problem or not.</pre>
<br />
<p>- jrose</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'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't found when it was RIGHT THERE, we can quickly tell them "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." And nine times out of ten they'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's an option to strip everything after a semicolon for extension matching. It's basically the same as the very first patch I made for this, but it seems like that'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>