kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-09-05 12:16:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Ensure bridge record error responses validate

Browse Source 
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
This commit is contained in:
Kinsey Moore
2013-10-15 20:03:19 +00:00
parent 6072e043cf
commit ec83706003
2 changed files with 17 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
res/res_ari_bridges.c
View File

@@ -742,6 +742,9 @@ static void ast_ari_record_bridge_cb(
break; break;
case 500: /* Internal Server Error */ case 500: /* Internal Server Error */
case 501: /* Not Implemented */ 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; is_valid = 1;
break; break;
default: default:

14
rest-api/api-docs/bridges.json
View File

@@ -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"
}
] ]
} }
] ]