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

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
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:                     confirmed
Asterisk Version:           Older 1.6.1 - please test a newer version 
JIRA:                        
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-07 11:31 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?

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

---------------------------------------------------------------------- 
 (0114850) fhackenberger (reporter) - 2009-12-07 11:31
 https://issues.asterisk.org/view.php?id=16284#c114850 
---------------------------------------------------------------------- 
New patch which fixes two bugs.

* I used the wrong channel to check for the AUTOMON flag
* I returned -1 in case the monitoring was fordiggen, which hung up the
channel

BTW the documentation of the builtin_automonitor function seems to be
wrong. It states:

 * \brief Monitor a channel by DTMF
 * \param chan channel requesting monitor
 * \param peer channel to be monitored

Scenario: 3951 calls 3950, 3950 hits *1
chan: SIP/3951-b3f134c0 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-12-07 11:31 fhackenberger  Note Added: 0114850                          
======================================================================