[asterisk-bugs] [Asterisk 0019248]: SRV lookup attempted for peers listed by IP address

[Asterisk Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=19248 
====================================================================== 
Reported By:                byronclark
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   19248
Category:                   Core/General
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:           1.8.3.3 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2011-05-05 15:44 CDT
Last Modified:              2011-05-05 15:44 CDT
====================================================================== 
Summary:                    SRV lookup attempted for peers listed by IP address
Description: 
Our configuration lists a lot of peers by IP address, like this:

[pbx](!)
type=peer
nat=yes
qualify=no
insecure=port,invite
deny=0.0.0.0/0.0.0.0
permit=10.199.7.0/255.255.255.0
call-limit=0

[10.199.7.1](pbx)
host=10.199.7.1

[10.199.7.2](pbx)
host=10.199.7.2


Upon upgrading from 1.6.2.17.3 to 1.8.3.3, asterisk took significantly
longer (minutes) to start. The startup delay comes because asterisk
attempts to do an SRV lookup for each of those peers which then times out.

The attached patch (against svn trunk) brings back a shortcut from
1.6.2.17.3 that skipped the SRV lookup if inet_aton succeeded on the host. 
Admittedly, the check should also handle an IPv6 address.
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-05-05 15:44 byronclark     New Issue                                    
2011-05-05 15:44 byronclark     Asterisk Version          => 1.8.3.3         
2011-05-05 15:44 byronclark     Regression                => No              
2011-05-05 15:44 byronclark     SVN Branch (only for SVN checkouts, not tarball
releases) => N/A             
======================================================================