[asterisk-dev] [Code Review] SIP Interop - Add an option to truncate user field at '; 's for the purpose of finding an extension.

[David Vossel]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1216/#review3543
-----------------------------------------------------------


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?

- David


On 2011-05-13 10:05:32, jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1216/
> -----------------------------------------------------------
> 
> (Updated 2011-05-13 10:05:32)
> 
> 
> Review request for Asterisk Developers, Russell Bryant, David Vossel, and Leif Madsen.
> 
> 
> Summary
> -------
> 
> 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.
> 
> 
> This addresses bug 18344.
>     https://issues.asterisk.org/view.php?id=18344
> 
> 
> Diffs
> -----
> 
>   /branches/1.8/channels/chan_sip.c 318634 
>   /branches/1.8/channels/sip/include/sip.h 318634 
>   /branches/1.8/configs/sip.conf.sample 318634 
> 
> Diff: https://reviewboard.asterisk.org/r/1216/diff
> 
> 
> Testing
> -------
> 
> 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
> 
> 
> Thanks,
> 
> jrose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110513/81564487/attachment-0001.htm>