[asterisk-bugs] [Asterisk 0008089]: [patch] The record# parameter in ENUMLOOKUP is ignored

noreply at bugs.digium.com noreply at bugs.digium.com
Thu May 1 11:36:46 CDT 2008


A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=8089 
====================================================================== 
Reported By:                otmar
Assigned To:                oej
====================================================================== 
Project:                    Asterisk
Issue ID:                   8089
Category:                   Functions/NewFeature
Reproducibility:            always
Severity:                   feature
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           1.6.0-beta5 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 50028 
Disclaimer on File?:        Yes 
Request Review:              
====================================================================== 
Date Submitted:             10-04-2006 10:19 CDT
Last Modified:              05-01-2008 11:36 CDT
====================================================================== 
Summary:                    [patch] The record# parameter in ENUMLOOKUP is
ignored
Description: 
In function_enum, the args.record parameter is never evaluated. 

That's also a bug in ast_get_enum, which does not offer a seperate
parameter for this.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0005526 [patch] Carrier ENUM support + some rew...
related to          0005666 [patch][post 1.4] Add NAPTR string pars...
related to          0011560 ENUM LOOKUP IS NOT WORKING AGAIN
====================================================================== 

---------------------------------------------------------------------- 
 bbryant - 05-01-08 11:36  
---------------------------------------------------------------------- 
otmar, it seems like your patch isn't backwards compatible with what is
currently in trunk making it unable to be merged.

Below are the results of my dialplan tests.

Testing with trunk...
[quote]
    -- Executing [2 at default:1] Answer("Zap/4-1", "") in new stack

    -- Executing [2 at default:2] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,,,,loligo.com)") in new stack
    -- Executing [2 at default:3] NoOp("Zap/4-1", "2203 at sip.fox-den.com") in
new stack

    -- Executing [2 at default:4] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,tel,,,loligo.com)") in new stack
    -- Executing [2 at default:5] NoOp("Zap/4-1", "+12125551212") in new
stack

    -- Executing [2 at default:6] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,sip,c,,loligo.com)") in new stack
    -- Executing [2 at default:7] NoOp("Zap/4-1", "3") in new stack

    -- Executing [2 at default:8] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,tel,,2,loligo.com)") in new stack
    -- Executing [2 at default:9] NoOp("Zap/4-1", "+12125551212") in new
stack

    -- Executing [2 at default:10] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,ALL,c,,loligo.com)") in new stack
    -- Executing [2 at default:11] NoOp("Zap/4-1", "6") in new stack

    -- Executing [2 at default:12] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,ALL,,2,loligo.com)") in new stack
    -- Executing [2 at default:13] NoOp("Zap/4-1", "tel:+12125551212") in new
stack

    -- Executing [2 at default:14] NoOp("Zap/4-1",
"ENUMLOOKUP(+437203001721)") in new stack
    -- Executing [2 at default:15] NoOp("Zap/4-1", "") in new stack

    -- Executing [2 at default:16] NoOp("Zap/4-1",
"ENUMLOOKUP(1234*256,,,,freenum.org)") in new stack
    -- Executing [2 at default:17] NoOp("Zap/4-1", "//www.loligo.com/") in
new stack

    -- Executing [2 at default:18] NoOp("Zap/4-1",
"ENUMLOOKUP(1234567890,sip,,1,enum.yoyodynelabs.com)") in new stack
    -- Executing [2 at default:19] NoOp("Zap/4-1", "") in new stack

    -- Executing [2 at default:20] Hangup("Zap/4-1", "") in new stack
[/quote]

Testing with trunk + patch....
[quote]
    -- Executing [2 at default:1] Answer("Zap/4-1", "") in new stack

    -- Executing [2 at default:2] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,,,,loligo.com)") in new stack
  	== ast_get_enum(num='+13015611020', tech='', suffix='loligo.com',
options='', record=1
  	== ENUM options(): pos=1, options='0'
    -- Executing [2 at default:3] NoOp("Zap/4-1", "") in new stack

    -- Executing [2 at default:4] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,tel,,,loligo.com)") in new stack
  	== ast_get_enum(num='+13015611020', tech='tel', suffix='loligo.com',
options='', record=1
  	== ENUM options(): pos=1, options='0'
    -- Executing [2 at default:5] NoOp("Zap/4-1", "") in new stack

    -- Executing [2 at default:6] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,sip,c,,loligo.com)") in new stack
  	== ast_get_enum(num='+13015611020', tech='sip', suffix='loligo.com',
options='c', record=1
  	== ENUM options(c): pos=1, options='1'
    -- Executing [2 at default:7] NoOp("Zap/4-1", "0") in new stack

    -- Executing [2 at default:8] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,tel,,2,loligo.com)") in new stack
  	== ast_get_enum(num='+13015611020', tech='tel', suffix='loligo.com',
options='', record=1
  	== ENUM options(): pos=1, options='0'
    -- Executing [2 at default:9] NoOp("Zap/4-1", "") in new stack

    -- Executing [2 at default:10] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,ALL,c,,loligo.com)") in new stack
  	== ast_get_enum(num='+13015611020', tech='ALL', suffix='loligo.com',
options='c', record=1
  	== ENUM options(c): pos=1, options='1'
    -- Executing [2 at default:11] NoOp("Zap/4-1", "0") in new stack

    -- Executing [2 at default:12] NoOp("Zap/4-1",
"ENUMLOOKUP(+13015611020,ALL,,2,loligo.com)") in new stack
  	== ast_get_enum(num='+13015611020', tech='ALL', suffix='loligo.com',
options='', record=1
  	== ENUM options(): pos=1, options='0'
    -- Executing [2 at default:13] NoOp("Zap/4-1", "") in new stack

    -- Executing [2 at default:14] NoOp("Zap/4-1",
"ENUMLOOKUP(+437203001721)") in new stack
  	== ast_get_enum(num='+437203001721', tech='sip', suffix='e164.arpa',
options='', record=1
  	== ENUM options(): pos=1, options='0'
    -- Executing [2 at default:15] NoOp("Zap/4-1", "") in new stack

    -- Executing [2 at default:16] NoOp("Zap/4-1",
"ENUMLOOKUP(1234*256,,,,freenum.org)") in new stack
  	== ast_get_enum(num='1234*256', tech='', suffix='freenum.org',
options='', record=1
  	== ENUM options(): pos=1, options='0'
    -- Executing [2 at default:17] NoOp("Zap/4-1", "") in new stack

    -- Executing [2 at default:18] NoOp("Zap/4-1",
"ENUMLOOKUP(1234567890,sip,,1,enum.yoyodynelabs.com)") in new stack
  	== ast_get_enum(num='1234567890', tech='sip',
suffix='enum.yoyodynelabs.com', options='', record=1
  	== ENUM options(): pos=1, options='0'
    -- Executing [2 at default:19] NoOp("Zap/4-1", "") in new stack

    -- Executing [2 at default:20] Hangup("Zap/4-1", "") in new stack
  	== Spawn extension (default, 2, 20) exited non-zero on 'Zap/4-1'
    -- Hungup 'Zap/4-1'
[/quote] 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
05-01-08 11:36  bbryant        Note Added: 0086256                          
======================================================================




More information about the asterisk-bugs mailing list