Review Board 1.7.16


ARI: Implement /recordings/stored API's

Review Request #2693 - Created July 23, 2013 and submitted

David Lee
/team/dlee/record
ASTERISK-21582
Reviewers
asterisk-dev
Asterisk
This patch implements the ARI API's for stored recordings. While the
original task only specified deleting a recording, it was simple
enough to implement the GET for all recordings, and for an individual
recording.

The recording playback operation was modified to use the same code for
accessing the recording as the REST API, so that they will behave
consistently.

There were several problems with the api-docs that were also fixed,
bringing the ARI spec in line with the implementation. There were some
'wishful thinking' fields on the stored recording model (duration and
timestamp) that were removed, because I ended up not implementing a
metadata file to go along with the recording to store such information.

The GET /recordings/live operation was removed, since it's not really
that useful to get a list of all recordings that are currently going
on in the system. (At least, if we did that, we'd probably want to
also list all of the current playbacks. Which seems weird.)
Used Swagger-UI to poke around the API. Verified that you couldn't
delete anything outside the recording's directory.
Total:
4
Open:
0
Resolved:
3
Dropped:
1
Status:
From:
Review request changed
Updated (Aug. 30, 2013, 9:21 a.m.)
  • changed from pending to submitted
Committed in revision 397999

https://reviewboard.asterisk.org/ runs on a server provided by Digium, Inc. and uses bandwidth donated to the open source Asterisk community by API Digital Communications in Huntsville, AL USA.
Please report problems with this site to asteriskteam@digium.com.