[asterisk-bugs] [Asterisk 0017796]: [patch] Allow chan_sip's support for INFO/Record request to also use dynamic features

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17796 
====================================================================== 
Reported By:                sircus
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   17796
Category:                   Channels/chan_sip/NewFeature
Reproducibility:            N/A
Severity:                   minor
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           SVN 
JIRA:                       SWP-2007 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 280909 
Request Review:              
====================================================================== 
Date Submitted:             2010-08-04 14:28 CDT
Last Modified:              2010-08-30 17:20 CDT
====================================================================== 
Summary:                    [patch] Allow chan_sip's support for INFO/Record
request to also use dynamic features
Description: 
chan_sip, in handle_request_info, handles INFO requests with a Record:
header, as sent by certain SIP phones (in my case, snoms) when the Record
button is pressed.  It does this by looking for application "automon" and
then faking the DTMF tones associated with it.

Since I want to implement somewhat different behaviour when the record
button is pressed, this behaviour is not useful for me.  The current code
doesn't look for dynamic features, so I have no route to customize the
behaviour.

The attached patches look for a dynamic feature if no built-in feature is
found, or no extension is assigned to the built-in feature.

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

---------------------------------------------------------------------- 
 (0126453) sircus (reporter) - 2010-08-30 17:20
 https://issues.asterisk.org/view.php?id=17796#c126453 
---------------------------------------------------------------------- 
Thanks for looking at the patch.  Whilst it's true that a sip.conf option
to choose the application to be used would be ideal, this patch is a
building block on the path in that direction.  

In order to be able to (sensibly) choose the app to run, you first need
chan_sip to be able to execute dynamic features - otherwise, you only get
to choose between the range of statically defined features. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-08-30 17:20 sircus         Note Added: 0126453                          
======================================================================