[asterisk-bugs] [Asterisk 0017291]: [patch] chan_iax2 ignores the port in an SRV record

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17291 
====================================================================== 
Reported By:                jcovert
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   17291
Category:                   Channels/chan_iax2
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.0.27 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.0 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-05-05 13:29 CDT
Last Modified:              2010-05-05 13:49 CDT
====================================================================== 
Summary:                    [patch] chan_iax2 ignores the port in an SRV record
Description: 
chan_iax2 ignores the port when doing an srv lookup.

The attached patch fixes this, and also turns srv lookup on by default.


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

---------------------------------------------------------------------- 
 (0121431) jsmith (administrator) - 2010-05-05 13:49
 https://issues.asterisk.org/view.php?id=17291#c121431 
---------------------------------------------------------------------- 
Just a quick comment, but I think you may want to split this up into two
separate patches.  I know they're both trivial, but we like making feature
changes independently from bug fixes.  Also, if you're doing to default to
doing SRV record lookups, you should also patch the iax.conf.sample file
and UPGRADE.txt files to make it obvious that we're changing behavior
here.

Other than that, the change looks fine to me. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-05-05 13:49 jsmith         Note Added: 0121431                          
======================================================================