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

Jeff Peeler jpeeler at digium.com
Tue Apr 20 16:47:38 CDT 2010


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

Ship it!


Pending the change on line 158, I think this is good to go!


trunk/include/asterisk/audiohook.h
<https://reviewboard.asterisk.org/r/487/#comment4033>

    Just a suggestion to be more clear:
    set or clear muted frames on direction based on flag parameter, flag parameter one line above could be more specific with respect to direction



trunk/main/audiohook.c
<https://reviewboard.asterisk.org/r/487/#comment4031>

    I believe you want to still compare the direction here too right?


- Jeff


On 2010-04-17 15:07:05, Julian Lyndon-Smith wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/487/
> -----------------------------------------------------------
> 
> (Updated 2010-04-17 15:07:05)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> 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 a new manager command
> 
> manager show command MixMonitorMute
> Action: MixMonitorMute
> [ActionID:] <value>
> Channel: <value>
> [Direction:] <value>
> [State:] <value>
> 
> [Synopsis]
> Mute / unMute a Mixmonitor recording.
> 
> [Description]
> This action may be used to mute a MixMonitor recording.
> 
> [Arguments]
> ActionID
>     ActionID for this transaction. Will be returned.
> Channel
>     Used to specify the channel to mute.
> Direction
>     Which part of the recording to mute:  read, write or both (from
>     channel, to channel or both channels).
> State
>     Turn mute on or off : 1 to turn on, 0 to turn off.
> 
> 
> This addresses bug 16740.
>     https://issues.asterisk.org/view.php?id=16740
> 
> 
> Diffs
> -----
> 
>   trunk/apps/app_mixmonitor.c 257640 
>   trunk/include/asterisk/audiohook.h 257640 
>   trunk/include/asterisk/frame.h 257640 
>   trunk/main/audiohook.c 257640 
>   trunk/main/frame.c 257640 
>   trunk/res/res_mutestream.c 257640 
> 
> Diff: https://reviewboard.asterisk.org/r/487/diff
> 
> 
> Testing
> -------
> 
> Tested on 1.4 : 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
> Tested on trunk : 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