[asterisk-dev] [Code Review] 2697: ARI: Add recording controls

svnbot reviewboard at asterisk.org
Tue Aug 6 09:44:52 CDT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2697/
-----------------------------------------------------------

(Updated Aug. 6, 2013, 9:44 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 396331


Bugs: ASTERISK-22181
    https://issues.asterisk.org/jira/browse/ASTERISK-22181


Repository: Asterisk


Description
-------

This patch implements the controls from ARI recordings. The controls
are:

 * DELETE /recordings/live/{recordingName} - stop recording and
   discard it
 * POST /recordings/live/{recordingName}/stop - stop recording
 * POST /recordings/live/{recordingName}/pause - pause recording
 * POST /recordings/live/{recordingName}/unpause - resume recording
 * POST /recordings/live/{recordingName}/mute - mute recording (record
   silence to the file)
 * POST /recordings/live/{recordingName}/unmute - unmute recording.

Since this underlying functionality did not already exist, is was
added to app.c by a set of control frames, similar to how playback
control works. The pause/mute control frames are toggles, even though
the ARI controls are idempotent, to be consistent with the playback
control frames.


Diffs
-----

  /trunk/apps/app_minivm.c 395927 
  /trunk/apps/app_voicemail.c 395927 
  /trunk/funcs/func_frame_trace.c 395927 
  /trunk/include/asterisk/app.h 395927 
  /trunk/include/asterisk/frame.h 395927 
  /trunk/include/asterisk/stasis_app_recording.h 395927 
  /trunk/main/app.c 395927 
  /trunk/main/channel.c 395927 
  /trunk/res/ari/resource_recordings.h 395927 
  /trunk/res/ari/resource_recordings.c 395927 
  /trunk/res/res_ari_recordings.c 395927 
  /trunk/res/res_stasis_recording.c 395927 
  /trunk/rest-api/api-docs/recordings.json 395927 

Diff: https://reviewboard.asterisk.org/r/2697/diff/


Testing
-------

Soooo many recordings.


Thanks,

David Lee

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130806/99026a9b/attachment.htm>


More information about the asterisk-dev mailing list