diff options
| author | Jonathan Rose <jrose@digium.com> | 2013-10-25 21:28:32 +0000 |
|---|---|---|
| committer | Jonathan Rose <jrose@digium.com> | 2013-10-25 21:28:32 +0000 |
| commit | d8a760307e43031033b5607264f7edd8ecfe73a4 (patch) | |
| tree | 782649b82cc96f634615d872044764497caec482 /res/res_ari_bridges.c | |
| parent | 7b42a6828a1f98862b673f9e1c2b9a5c105231db (diff) | |
ARI recordings: Issue HTTP failures for recording requests with file conflicts
If a file already exists in the recordings directory with the same name as what
we would record, issue a 422 instead of relying on the internal failure and
issuing success.
(closes issue ASTERISK-22623)
Reported by: Joshua Colp
Review: https://reviewboard.asterisk.org/r/2922/
........
Merged revisions 401973 from http://svn.asterisk.org/svn/asterisk/branches/12
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@401999 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'res/res_ari_bridges.c')
| -rw-r--r-- | res/res_ari_bridges.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/res/res_ari_bridges.c b/res/res_ari_bridges.c index 3908094a8..f6a3c1c1a 100644 --- a/res/res_ari_bridges.c +++ b/res/res_ari_bridges.c @@ -742,9 +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 400: /* Invalid parameters */ case 404: /* Bridge not found */ - case 409: /* Bridge not in Stasis application; Recording already in progress */ + case 409: /* Bridge is not in a Stasis application; A recording with the same name already exists on the system and can not be overwritten because it is in progress or ifExists=fail */ is_valid = 1; break; default: |