[asterisk-dev] [Code Review] 3840: bridge: Move "bridge destroy" CLI command to devmode and add "all" to "bridge kick" CLI command

Joshua Colp reviewboard at asterisk.org
Thu Jul 24 16:51:20 CDT 2014


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

(Updated July 24, 2014, 9:51 p.m.)


Review request for Asterisk Developers.


Repository: Asterisk


Description
-------

The "bridge destroy" CLI command is dangerous for mere mortals. It can send bridges into an unusable state when the owner of them did not expect it to happen. Its very existence instills paranoia upon the users of bridges since they must be aware that the very bridge they created may need to be recreated at any time. I think this is unreasonable so I've moved it to be a devmode only CLI command since it can potentially be useful when working with things. In its normal place I have added "all" as a valid option to the "bridge kick" CLI command. When invoked all channels will be kicked out of the bridge instead of one specific one.


Diffs (updated)
-----

  /branches/12/main/bridge.c 419341 

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


Testing
-------

Executed "bridge kick all" and confirmed all channels were kicked out.


Thanks,

Joshua Colp

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


More information about the asterisk-dev mailing list