[asterisk-dev] [Code Review] 3211: ARI: URI is case sensitive

Scott Griepentrog reviewboard at asterisk.org
Tue Feb 11 19:09:03 CST 2014

This is an automatically generated e-mail. To reply, visit:

Review request for Asterisk Developers.

Bugs: ASTERISK-23125

Repository: Asterisk


URI's are supposed to be case sensitive, and traditionally all lower case for arbitrary path names.  However, the Asterisk convention of CAPS for technology names (SIP, PJSIP, LOCAL, etc) was causing a situation where certain ARI URI's would work in either case, and others not.

The use of all lower case in ARI URI's, including technology names, is desired.  This patch causes TECH names to be uppercase before lookup in stasis.

Note that I have chosen to not enforce lower case on the /ari prefix portion of the URI at this time, in case it would cause something else to break.

Reference for asterisk-app-dev list discussion: http://lists.digium.com/pipermail/asterisk-app-dev/2014-January/000321.html


  /branches/12/main/stasis_endpoints.c 407957 

Diff: https://reviewboard.asterisk.org/r/3211/diff/


Checked that /ari/endpoints/PJSIP/{endpoint} and /ari/endpoints/pjsip/{endpoint} both work.


Scott Griepentrog

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140212/1c156a74/attachment.html>

More information about the asterisk-dev mailing list