diff options
author | Scott Griepentrog <sgriepentrog@digium.com> | 2014-02-14 21:44:57 +0000 |
---|---|---|
committer | Scott Griepentrog <sgriepentrog@digium.com> | 2014-02-14 21:44:57 +0000 |
commit | 04fe3bccc0d89fb25cba4275337384602930b6d7 (patch) | |
tree | 53499334b5f55d2e500c7bbcbcdc9b4fa9e430f1 | |
parent | c41040fd4b9a4f14bbd0be30500361154d823595 (diff) |
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
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@408141 65c4cc65-6c06-0410-ace0-fbb531ad65f3
-rw-r--r-- | main/stasis_endpoints.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/main/stasis_endpoints.c b/main/stasis_endpoints.c index 81c4f15a3..3f8d32419 100644 --- a/main/stasis_endpoints.c +++ b/main/stasis_endpoints.c @@ -197,6 +197,7 @@ struct ast_endpoint_snapshot *ast_endpoint_latest_snapshot(const char *tech, if (!id) { return NULL; } + ast_tech_to_upper(id); msg = stasis_cache_get(ast_endpoint_cache(), ast_endpoint_snapshot_type(), id); |