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

[jrose]

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

(Updated 2011-05-13 10:05:32.508198)


Review request for Asterisk Developers, Russell Bryant, David Vossel, and Leif Madsen.


Changes
-------

Fixed a misworded statement.


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 (updated)
-------

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/5e38db62/attachment.htm>