[asterisk-bugs] [Asterisk 0016284]: [patch] Support for disabling automon selectively per peer

[Asterisk Bug Tracker]

The following issue requires your FEEDBACK. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16284 
====================================================================== 
Reported By:                fhackenberger
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   16284
Category:                   Applications/app_mixmonitor
Reproducibility:            always
Severity:                   tweak
Priority:                   normal
Status:                     feedback
Asterisk Version:           Older 1.6.1 - please test a newer version 
JIRA:                       SWP-496 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-11-19 06:24 CST
Last Modified:              2009-12-08 10:39 CST
====================================================================== 
Summary:                    [patch] Support for disabling automon selectively
per peer
Description: 
chan_sip.c states:
 OEJ: I think this should be something that is enabled/disabled
 per device. I don't want incoming callers to record calls in my
 pbx.

I propose the attached implementation. It adds a new channel option
AST_OPTION_ALLOW_AUTOMON. Every channel can then respond to a query for
that option. As an example, I implemented support for an allowautomon
setting for sip peers.

Would that be a viable way to implement the tweak?

====================================================================== 

---------------------------------------------------------------------- 
 (0114918) russell (administrator) - 2009-12-08 10:39
 https://issues.asterisk.org/view.php?id=16284#c114918 
---------------------------------------------------------------------- 
Thank you very much for the contribution.

However, I do not think this patch is necessary.  There are already tools
in place to allow this.  Specifically, chan_sip allows variables to be set
on channels from your configuration.  (The setvar option in sip.conf).