[svn-commits] sgriepentrog: trunk r408141 - in /trunk: ./ main/stasis_endpoints.c

[SVN commits to the Digium repositories]

Author: sgriepentrog
Date: Fri Feb 14 15:44:57 2014
New Revision: 408141

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=408141
Log:
ARI: correct upper/lower case URI discrepancies

URI's are supposed to be case sensitive and all
lower case.  In practice some portions of URI's
in ARI are case insensitive and others are not,
such as TECH, which in one instance would match
a lower case name and in another would not.  In
this patch, the ast_endpoint_lastest_snapshot()
function is modified to change the TECH portion
to full upper case before lookup. This resolves
the discrepancy noted by the reporter.  However
I chose to avoid forcing the /ari prefix of the
URI's to be lower case for now.  Except for the
two cases here, all URI's should be lower case,
unless they are part of a resource name or id.

Review: https://reviewboard.asterisk.org/r/3211/
Reported by: Zane Conkle
(closes issue ASTERISK-23125)
........

Merged revisions 408140 from http://svn.asterisk.org/svn/asterisk/branches/12

Modified:
    trunk/   (props changed)
    trunk/main/stasis_endpoints.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-12-merged' - no diff available.

Modified: trunk/main/stasis_endpoints.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/stasis_endpoints.c?view=diff&rev=408141&r1=408140&r2=408141
==============================================================================
--- trunk/main/stasis_endpoints.c (original)
+++ trunk/main/stasis_endpoints.c Fri Feb 14 15:44:57 2014
@@ -197,6 +197,7 @@
 	if (!id) {
 		return NULL;
 	}
+	ast_tech_to_upper(id);
 
 	msg = stasis_cache_get(ast_endpoint_cache(),
 		ast_endpoint_snapshot_type(), id);