[asterisk-bugs] [Asterisk 0015809]: [patch] callprogress and faxdetect not read when building channel results in fax detection failure

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15809 
====================================================================== 
Reported By:                phsultan
Assigned To:                kpfleming
====================================================================== 
Project:                    Asterisk
Issue ID:                   15809
Category:                   Channels/chan_dahdi
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
Target Version:             1.6.1.x Pending Blocker
Asterisk Version:           Older 1.6.1 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-09-01 13:15 CDT
Last Modified:              2009-09-01 16:59 CDT
====================================================================== 
Summary:                    [patch] callprogress and faxdetect not read when
building channel results in fax detection failure
Description: 
The faxdetect and callprogress attributes are not read when creating
channels with build_channels with channels/chan_dahdi.c, and therefore, the
following chan_dahdi.conf prevents fax detection from happening :
[channels]
#include "dahdi-channels.conf"
callprogress=yes
faxdetect=both
usecallerid=yes
callwaiting=yes
usecallingpres=yes
callwaitingcallerid=yes
threewaycalling=yes
transfer=yes
canpark=yes
cancallforward=yes
callreturn=yes
echocancel=no

Actually, it looks like the whole [channels] section should be read to
populate the conf variable before calling build_channels().

I'm not sure if it's the right way to solve this issue, and there is a
configuration workaround (see the additional information). The attached
patch does the following :
- make Asterisk read the [channels] section without building channels
first ;
- build a first version of the configuration in the base_conf variable ;
- build a new configuration from base_conf and actually call
build_channels() after reading the channel attribute.

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

---------------------------------------------------------------------- 
 (0109976) phsultan (manager) - 2009-09-01 16:59
 https://issues.asterisk.org/view.php?id=15809#c109976 
---------------------------------------------------------------------- 
Ok I understand. I found several configuration snippets that set the
#include dahdi-channels.conf statement before all the attributes under the
[channels] category, and spent half a day to understand why I couldn't have
the fax detection working.

Perhaps a explicit note in dahdi-channels.conf mentioning this file should
be included after the attributes under [channels] can be of any help? At
least it would have been for people like me :)

Thanks for the clarification Kevin, please close the ticket. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-09-01 16:59 phsultan       Note Added: 0109976                          
======================================================================