[asterisk-dev] [Code Review] Mute / Unmute manager commands for MixMonitor

Julian Lyndon-Smith asterisk at dotr.com
Tue Feb 2 17:09:04 CST 2010


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

(Updated 2010-02-02 17:09:04.653132)


Review request for Asterisk Developers.


Changes
-------

added missing "f", changed "is" to "in"


Summary (updated)
-------

For PCI-DSS compliance we are not allowed to record a credit card number in a MixMonitor file. However, we must record all conversations ....

I have added a new feature to audiohooks so that you can mute either read / write or both types of frames - this allows for MixMonitor to mute either side of the conversation without affecting the conversation itself.

MixMonitor now has two manager commands

1) manager show command MuteMixMonitor
Action: MuteMixMonitor
Synopsis: Mute a channel in MixMonitor
Privilege: <none>
Description: Mutes a Mixmonitor Channel.
Variables:
  Channel: Channel to mute.
  Direction: Which part to mute. read|write|both (from channel|to channel|both channels).

2) manager show command unMuteMixMonitor
Action: unMuteMixMonitor
Synopsis: unMute a channel in MixMonitor
Privilege: <none>
Description: unMutes a Mixmonitor Channel.
Variables:
  Channel: Channel to unmute.
  Direction: Which part to unmute. read|write|both (from channel|to channel|both channels).


This addresses bug 16740.
    https://issues.asterisk.org/view.php?id=16740


Diffs
-----

  trunk/apps/app_mixmonitor.c 244049 
  trunk/include/asterisk/audiohook.h 244049 
  trunk/main/audiohook.c 244049 

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


Testing
-------

Tested on a 1.4 version (only difference I think is that frame->data in 1.4 becomes frame->data.ptr in trunk in audiohook.c

Tested muting each direction, and both directions, and unmuting, and then listening to the mixmonitor file. The mutes and unmutes all seemed to be in the right place at the right time


Thanks,

Julian




More information about the asterisk-dev mailing list