<div dir="ltr"><div class="gmail_default" style="color:#660000">I was assuming that we would leave TECH to be case INsensitive, and thus it wouldn't matter. We can also then optionally go through and change all json output to lowercase.</div>
<div class="gmail_default" style="color:#660000"><br></div><div class="gmail_default" style="color:#660000">If you want to treat TECH as a case sensitive value, then ALL instances of IAX and PJSIP and LOCAL etc would have to be changed everywhere in the code (for any json output anyway) so that you don't have code broken by a lowercase TECH requirement. This would also break some EXISTING ari apps, likely also tests.</div>
<div class="gmail_default" style="color:#660000"><br></div><div class="gmail_default" style="color:#660000">What about a transitional period (such as Asterisk 12) where TECH is case insensitive and the json output of TECH is lowercased, then later (in trunk & Asterisk 13) the case sensitivity on TECH can be changed?</div>
<div class="gmail_default" style="color:#660000"><br></div><div class="gmail_default" style="color:#660000"><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 21, 2014 at 2:50 PM, Paul Belanger <span dir="ltr"><<a href="mailto:paul.belanger@polybeacon.com" target="_blank">paul.belanger@polybeacon.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Jan 21, 2014 at 3:47 PM, Scott Griepentrog<br>
<<a href="mailto:sgriepentrog@digium.com">sgriepentrog@digium.com</a>> wrote:<br>
><br>
> Would you extend TECH being lower case to replacing all cases where PJSIP is output in response to an ARI request (and thus could get copied into a URL)? For example, instead of:<br>
><br>
> {<br>
> "technology": "IAX2",<br>
> "resource": "demo",<br>
> "state": "unknown",<br>
> "channel_ids": []<br>
> },<br>
> {<br>
> "technology": "PJSIP",<br>
> "resource": "200",<br>
> "state": "offline",<br>
> "channel_ids": []<br>
> },<br>
><br>
> You would want:<br>
><br>
> {<br>
> "technology": "iax2",<br>
> "resource": "demo",<br>
> "state": "unknown",<br>
> "channel_ids": []<br>
> },<br>
> {<br>
> "technology": "pjsip",<br>
> "resource": "200",<br>
> "state": "offline",<br>
> "channel_ids": []<br>
> },<br>
><br>
> Or would it be sufficient to say that lowercase is preferred, but due to case insensitivity it's not necessary to change all existing cases where it is output?<br>
><br>
</div></div>Right, I would expect only lower case TECH to work after the change.<br>
URL are case sensitive, but we are forcing lower case for TECH.<br>
<br>
So:<br>
<br>
/iax2/demo works!<br>
<br>
and<br>
<br>
/IAX2/demo 404<br>
<div class="im HOEnZb"><br>
--<br>
Paul Belanger | PolyBeacon, Inc.<br>
Jabber: <a href="mailto:paul.belanger@polybeacon.com">paul.belanger@polybeacon.com</a> | IRC: pabelanger (Freenode)<br>
Github: <a href="https://github.com/pabelanger" target="_blank">https://github.com/pabelanger</a> | Twitter: <a href="https://twitter.com/pabelanger" target="_blank">https://twitter.com/pabelanger</a><br>
<br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
asterisk-app-dev mailing list<br>
<a href="mailto:asterisk-app-dev@lists.digium.com">asterisk-app-dev@lists.digium.com</a><br>
<a href="http://lists.digium.com/cgi-bin/mailman/listinfo/asterisk-app-dev" target="_blank">http://lists.digium.com/cgi-bin/mailman/listinfo/asterisk-app-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><img alt="Digium logo" src="https://my.digium.com/images/graphics/digium_RGB_signature.gif" width="288" height="50" style="color:rgb(0,0,0);font-family:Arial,Helvetica,sans-serif;font-size:12px"><div>
Scott Griepentrog<br>Digium, Inc · Software Developer<br>445 Jan Davis Drive NW · Huntsville, AL 35806 · US<br>direct/fax: +1 256 428 6239 · mobile: +1 317 507 4029<br>Check us out at: <a href="http://digium.com" target="_blank">http://digium.com</a> · <a href="http://asterisk.org" target="_blank">http://asterisk.org</a><br>
</div></div>
</div>