[asterisk-dev] [Code Review] 2947: ARI: Adding a channel to a bridge while a live recording is active blocks

Kevin Harwell reviewboard at asterisk.org
Fri Nov 15 12:23:39 CST 2013


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

(Updated Nov. 15, 2013, 12:23 p.m.)


Review request for Asterisk Developers.


Changes
-------

Addressed review findings.


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


Repository: Asterisk


Description
-------

Added a channel recording flag that is checked before adding a channel to a bridge.  If the the channel is currently being recorded a "409 Conflict" is returned and the channel is not added to the bridge.


Diffs (updated)
-----

  branches/12/rest-api/api-docs/bridges.json 402853 
  branches/12/res/stasis/control.c 402853 
  branches/12/res/stasis/command.c 402853 
  branches/12/res/stasis/command.h 402853 
  branches/12/res/res_stasis_recording.c 402853 
  branches/12/res/res_stasis_playback.c 402853 
  branches/12/res/res_stasis_answer.c 402853 
  branches/12/res/res_ari_bridges.c 402853 
  branches/12/res/ari/resource_bridges.c 402853 
  branches/12/include/asterisk/stasis_app_impl.h 402853 
  branches/12/include/asterisk/stasis_app.h 402853 

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


Testing
-------

Ran a manual test and also wrote a testsuite test: https://reviewboard.asterisk.org/r/2951/


Thanks,

Kevin Harwell

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20131115/db68cbe9/attachment.html>


More information about the asterisk-dev mailing list