diff options
author | Kinsey Moore <kmoore@digium.com> | 2013-10-15 20:03:19 +0000 |
---|---|---|
committer | Kinsey Moore <kmoore@digium.com> | 2013-10-15 20:03:19 +0000 |
commit | ec837060038cff524e4b62536b38efead80553bf (patch) | |
tree | d287571359faf70b3d0a8a5ea1749f8e0d5d523d | |
parent | 6072e043cfd032a85fc12a7aa772684baac5b44b (diff) |
Ensure bridge record error responses validate
This adds the list of expected errors to the /bridges/{bridgeId}/record
ARI documentation so that outbound 4xx errors validate properly.
Previously, this would result in a response validation failure.
(closes issue ASTERISK-22627)
Reported by: Joshua Colp
........
Merged revisions 401018 from http://svn.asterisk.org/svn/asterisk/branches/12
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@401019 65c4cc65-6c06-0410-ace0-fbb531ad65f3
-rw-r--r-- | res/res_ari_bridges.c | 3 | ||||
-rw-r--r-- | rest-api/api-docs/bridges.json | 14 |
2 files changed, 17 insertions, 0 deletions
diff --git a/res/res_ari_bridges.c b/res/res_ari_bridges.c index d3b3a649d..c0b8b0d58 100644 --- a/res/res_ari_bridges.c +++ b/res/res_ari_bridges.c @@ -742,6 +742,9 @@ static void ast_ari_record_bridge_cb( break; case 500: /* Internal Server Error */ case 501: /* Not Implemented */ + case 400: /* Recording name invalid */ + case 404: /* Bridge not found */ + case 409: /* Bridge not in Stasis application; Recording already in progress */ is_valid = 1; break; default: diff --git a/rest-api/api-docs/bridges.json b/rest-api/api-docs/bridges.json index 640cf4c5d..48e5696c5 100644 --- a/rest-api/api-docs/bridges.json +++ b/rest-api/api-docs/bridges.json @@ -453,6 +453,20 @@ ] } } + ], + "errorResponses": [ + { + "code": 400, + "reason": "Recording name invalid" + }, + { + "code": 404, + "reason": "Bridge not found" + }, + { + "code": 409, + "reason": "Bridge not in Stasis application; Recording already in progress" + } ] } ] |