diff options
author | Kevin Harwell <kharwell@digium.com> | 2013-11-12 22:17:28 +0000 |
---|---|---|
committer | Kevin Harwell <kharwell@digium.com> | 2013-11-12 22:17:28 +0000 |
commit | 4be01b4598f37431809980907d1ab7ce02d0e47a (patch) | |
tree | 290c7d55c8a83597ddb6a968768510125458212d | |
parent | 94f19c82184bf508adc3f5e2398d0a776491b53c (diff) |
ari endpoints: GET /ari/endpoints/{invalid-tech} should return a 404
Implementation listing endpoints by technology returned an empty array if no
matching endpoints were found. Fixed so a "404 Not Found" will be returned
instead.
(closes issue ASTERISK-22803)
Reported by: David M. Lee
........
Merged revisions 402787 from http://svn.asterisk.org/svn/asterisk/branches/12
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@402788 65c4cc65-6c06-0410-ace0-fbb531ad65f3
-rw-r--r-- | res/ari/resource_endpoints.c | 9 | ||||
-rw-r--r-- | res/res_ari_endpoints.c | 2 | ||||
-rw-r--r-- | rest-api/api-docs/endpoints.json | 12 |
3 files changed, 20 insertions, 3 deletions
diff --git a/res/ari/resource_endpoints.c b/res/ari/resource_endpoints.c index 181ce7a50..e08f6e559 100644 --- a/res/ari/resource_endpoints.c +++ b/res/ari/resource_endpoints.c @@ -89,8 +89,6 @@ void ast_ari_endpoints_list_by_tech(struct ast_variable *headers, struct ao2_iterator i; void *obj; - /* TODO - if tech isn't a recognized type of endpoint, it should 404 */ - cache = ast_endpoint_cache(); if (!cache) { ast_ari_response_error( @@ -131,7 +129,12 @@ void ast_ari_endpoints_list_by_tech(struct ast_variable *headers, } ao2_iterator_destroy(&i); - ast_ari_response_ok(response, ast_json_ref(json)); + if (ast_json_array_size(json)) { + ast_ari_response_ok(response, ast_json_ref(json)); + } else { + ast_ari_response_error(response, 404, "Not Found", + "No Endpoints found with tech %s", args->tech); + } } void ast_ari_endpoints_get(struct ast_variable *headers, struct ast_ari_endpoints_get_args *args, diff --git a/res/res_ari_endpoints.c b/res/res_ari_endpoints.c index 082272518..10accb968 100644 --- a/res/res_ari_endpoints.c +++ b/res/res_ari_endpoints.c @@ -134,6 +134,7 @@ static void ast_ari_endpoints_list_by_tech_cb( break; case 500: /* Internal Server Error */ case 501: /* Not Implemented */ + case 404: /* Endpoints not found */ is_valid = 1; break; default: @@ -193,6 +194,7 @@ static void ast_ari_endpoints_get_cb( break; case 500: /* Internal Server Error */ case 501: /* Not Implemented */ + case 404: /* Endpoints not found */ is_valid = 1; break; default: diff --git a/rest-api/api-docs/endpoints.json b/rest-api/api-docs/endpoints.json index c44e3dd7e..fb66357a7 100644 --- a/rest-api/api-docs/endpoints.json +++ b/rest-api/api-docs/endpoints.json @@ -35,6 +35,12 @@ "paramType": "path", "dataType": "string" } + ], + "errorResponses": [ + { + "code": 404, + "reason": "Endpoints not found" + } ] } ] @@ -61,6 +67,12 @@ "paramType": "path", "dataType": "string" } + ], + "errorResponses": [ + { + "code": 404, + "reason": "Endpoints not found" + } ] } ] |