[Home]

Summary:ASTERISK-14799: [patch] Revision 152765 introduces scoping difficulties in stdexten
Reporter:Philip Prindeville (pprindeville)Labels:
Date Opened:2009-09-08 18:59:47Date Closed:2009-11-03 13:31:57.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/Configuration
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) extensions.conf
( 1) stdexten-context-update.txt
Description:The new [stdexten] macro doesn't always seem to be properly scoped.

Testing the redfish config below, I get:

...
   -- Executing [s@redfish-pstn:27] Gosub("SIP/sip_proxy-0847e6d8", "stdexten(111,SIP/bedroom_1&SIP/office_2&SIP/remote_2&SIP/kitchen_1&Local/2088919324@follow-me&SIP/softphone,redfish)") in new stack
[Sep  8 17:34:14] NOTICE[7503]: pbx.c:3233 pbx_extension_helper: No such label 'stdexten' in extension 's' in context 'redfish-pstn'
[Sep  8 17:34:14] WARNING[7503]: pbx.c:8998 pbx_parseable_goto: Priority 'stdexten' must be a number > 0, or valid label
[Sep  8 17:34:14] ERROR[7503]: app_stack.c:279 gosub_exec: Gosub address is invalid: 'stdexten(111,SIP/bedroom_1&SIP/office_2&SIP/remote_2&SIP/kitchen_1&Local/2088919324@follow-me&SIP/softphone,redfish)'
 == Spawn extension (redfish-pstn, s, 27) exited non-zero on 'SIP/sip_proxy-0847e6d8'

on externally originating calls, even though it seems to work for internal calls.

Both the [redfish-internal] and the [redfish-pstn] contexts include the [redfish-extens] context.

Reverting part of the context to revision 147896 I once again see success:

...
   -- Executing [s@redfish-pstn:27] Gosub("SIP/sip_proxy-08493c08", "stdexten,s,1(111,SIP/bedroom_1&SIP/office_2&SIP/remote_2&SIP/kitchen_1&Local/2088919324@follow-me&SIP/softphone,redfish)") in new stack
   -- Executing [s@stdexten:1] NoOp("SIP/sip_proxy-08493c08", "Start stdexten") in new stack
...



Comments:By: Philip Prindeville (pprindeville) 2009-09-08 19:15:31

Attaching the relevant parts of the extensions.conf file, with some elisions indicated as "..."
By: Leif Madsen (lmadsen) 2009-09-15 13:00:18

This is a duplicate of 15857, so I'm going to close this one out. Thanks!
By: Philip Prindeville (pprindeville) 2009-11-03 12:33:46.000-0600

Please undupe this.

15857 was resolved, but only the issues in 15857 were addressed.

15858 is still outstanding.
By: Leif Madsen (lmadsen) 2009-11-03 12:52:39.000-0600

I still have absolutely no idea why this is an issue. I see all the dialplan going through and it should work fine, unless it can't search for the 'stdexten' priority in nested includes. Regardless, this looks to be a configuration problem to me.
By: Leif Madsen (lmadsen) 2009-11-03 13:08:18.000-0600

Try this.
By: Digium Subversion (svnbot) 2009-11-03 13:30:42.000-0600

Repository: asterisk
Revision: 227361

U   trunk/configs/extensions.conf.sample

------------------------------------------------------------------------
r227361 | lmadsen | 2009-11-03 13:30:41 -0600 (Tue, 03 Nov 2009) | 11 lines

Additional fixes to the extensions.conf.sample file.

Update the extensions.conf.sample [stdexten] context so that we use the
variable instead of requiring it to be passed explicitly. Also updated uses of
the [stdexten] context throughout.

(closes issue ASTERISK-14799)
Reported by: pprindeville
Patches:
     stdexten-context-update.txt uploaded by lmadsen (license 10)
Tested by: pprindeville
------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=227361
By: Digium Subversion (svnbot) 2009-11-03 13:31:10.000-0600

Repository: asterisk
Revision: 227362

_U  branches/1.6.0/
U   branches/1.6.0/configs/extensions.conf.sample

------------------------------------------------------------------------
r227362 | lmadsen | 2009-11-03 13:31:09 -0600 (Tue, 03 Nov 2009) | 19 lines

Merged revisions 227361 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r227361 | lmadsen | 2009-11-03 13:25:18 -0600 (Tue, 03 Nov 2009) | 11 lines
 
 Additional fixes to the extensions.conf.sample file.
 
 Update the extensions.conf.sample [stdexten] context so that we use the
 variable instead of requiring it to be passed explicitly. Also updated uses of
 the [stdexten] context throughout.
 
 (closes issue ASTERISK-14799)
 Reported by: pprindeville
 Patches:
       stdexten-context-update.txt uploaded by lmadsen (license 10)
 Tested by: pprindeville
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=227362
By: Digium Subversion (svnbot) 2009-11-03 13:31:37.000-0600

Repository: asterisk
Revision: 227363

_U  branches/1.6.1/
U   branches/1.6.1/configs/extensions.conf.sample

------------------------------------------------------------------------
r227363 | lmadsen | 2009-11-03 13:31:36 -0600 (Tue, 03 Nov 2009) | 19 lines

Merged revisions 227361 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r227361 | lmadsen | 2009-11-03 13:25:18 -0600 (Tue, 03 Nov 2009) | 11 lines
 
 Additional fixes to the extensions.conf.sample file.
 
 Update the extensions.conf.sample [stdexten] context so that we use the
 variable instead of requiring it to be passed explicitly. Also updated uses of
 the [stdexten] context throughout.
 
 (closes issue ASTERISK-14799)
 Reported by: pprindeville
 Patches:
       stdexten-context-update.txt uploaded by lmadsen (license 10)
 Tested by: pprindeville
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=227363
By: Digium Subversion (svnbot) 2009-11-03 13:31:56.000-0600

Repository: asterisk
Revision: 227364

_U  branches/1.6.2/
U   branches/1.6.2/configs/extensions.conf.sample

------------------------------------------------------------------------
r227364 | lmadsen | 2009-11-03 13:31:56 -0600 (Tue, 03 Nov 2009) | 19 lines

Merged revisions 227361 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r227361 | lmadsen | 2009-11-03 13:25:18 -0600 (Tue, 03 Nov 2009) | 11 lines
 
 Additional fixes to the extensions.conf.sample file.
 
 Update the extensions.conf.sample [stdexten] context so that we use the
 variable instead of requiring it to be passed explicitly. Also updated uses of
 the [stdexten] context throughout.
 
 (closes issue ASTERISK-14799)
 Reported by: pprindeville
 Patches:
       stdexten-context-update.txt uploaded by lmadsen (license 10)
 Tested by: pprindeville
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=227364