diff options
author | Joshua Colp <jcolp@digium.com> | 2015-02-21 20:48:17 +0000 |
---|---|---|
committer | Joshua Colp <jcolp@digium.com> | 2015-02-21 20:48:17 +0000 |
commit | bedf51b2cebab8284aafa20237c883e9db6f6bb2 (patch) | |
tree | a57047b299e363a1bb5ad777437c698fdf1cc2e7 /res/ari | |
parent | 87b7060f3657f00e32d8da3673fa407157e5eebf (diff) |
res_ari_channels: Return a 404 response when a requested channel variable does not exist.
This change makes it so that if a channel variable is requested and it does not exist
a 404 response will be returned instead of an allocation failed response. This makes
it easier to debug and figure out what is going on for a user.
ASTERISK-24677 #close
Reported by: Joshua Colp
........
Merged revisions 432154 from http://svn.asterisk.org/svn/asterisk/branches/13
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@432155 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'res/ari')
-rw-r--r-- | res/ari/resource_channels.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/res/ari/resource_channels.c b/res/ari/resource_channels.c index 67498af9f..edd319d37 100644 --- a/res/ari/resource_channels.c +++ b/res/ari/resource_channels.c @@ -1242,6 +1242,11 @@ void ast_ari_channels_get_channel_var(struct ast_variable *headers, ast_assert(response != NULL); + if (!value) { + ast_ari_response_alloc_failed(response); + return; + } + if (ast_strlen_zero(args->variable)) { ast_ari_response_error( response, 400, "Bad Request", @@ -1280,7 +1285,9 @@ void ast_ari_channels_get_channel_var(struct ast_variable *headers, } } else { if (!ast_str_retrieve_variable(&value, 0, channel, NULL, args->variable)) { - ast_ari_response_alloc_failed(response); + ast_ari_response_error( + response, 404, "Variable Not Found", + "Provided variable was not found"); return; } } |