[asterisk-bugs] [Asterisk 0019136]: [patch] 'e' special extension fails to trigger in at least two cases

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=19136 
====================================================================== 
Reported By:                kshumard
Assigned To:                rmudgett
====================================================================== 
Project:                    Asterisk
Issue ID:                   19136
Category:                   PBX/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
Target Version:             1.8.5
Asterisk Version:           SVN 
JIRA:                       SWP-3347 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!): 314115 
Request Review:              
====================================================================== 
Date Submitted:             2011-04-18 14:45 CDT
Last Modified:              2011-04-22 18:31 CDT
====================================================================== 
Summary:                    [patch] 'e' special extension fails to trigger in at
least two cases
Description: 
https://wiki.asterisk.org/wiki/display/AST/The+Asterisk+Dialplan says, "e -
This extension will substitute as a catchall for any of the 'i', 't', or
'T' extensions, if any of them do not exist and catching the error in a
single routine is desired. The function EXCEPTION may be used to query the
type of exception or the location where it occurred."

The attached patch attempts to make this statement more true. In testing,
neither 'i' nor 't' was present but when the appropriate conditions were
met the channel didn't move it to the 'e' extension. The patch throws an
exception if WaitExten results in a timeout, and explicitly jumps to e,1
when an invalid extension is entered and there's no 'i' (though the
proposed patch might not be the proper way to accomplish this).

pbx_builtin_background should also throw an exception if it times out, but
currently doesn't. I just didn't know the appropriate logic to use at the
end of the function to add it in.
====================================================================== 

---------------------------------------------------------------------- 
 (0134056) rmudgett (administrator) - 2011-04-22 18:31
 https://issues.asterisk.org/view.php?id=19136#c134056 
---------------------------------------------------------------------- 
The issue19136_v1.8.patch contains the fixes in pbx.c.e-patch as well as
several more items I found for the 'e' extension.

The Background application should not be raising a timeout exception
because:
1) It changes behavior.
2) If a timeout happens the dialplan continues executing with the next
priority.  I think this is more desirable than an exception anyway. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-04-22 18:31 rmudgett       Note Added: 0134056                          
======================================================================