[asterisk-bugs] [Asterisk 0014321]: The contact exten field in the sip.conf register string is not parsed

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14321 
====================================================================== 
Reported By:                Nick_Lewis
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   14321
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.1-beta4 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-01-23 07:58 CST
Last Modified:              2009-01-26 03:21 CST
====================================================================== 
Summary:                    The contact exten field in the sip.conf register
string is not parsed
Description: 
The contact exten field for all registrations is set to "s" irrespective of
the value of the contact exten parameter in the register string.

This problem was first mentioned in note 97149 of issue 14185. I think
that the patches that address the primary problem with expiry in issue
14185 also address this contact exten issue. However if a separate patch is
needed for this alone then I can make one.
====================================================================== 

---------------------------------------------------------------------- 
 (0098722) Nick_Lewis (reporter) - 2009-01-26 03:21
 http://bugs.digium.com/view.php?id=14321#c98722 
---------------------------------------------------------------------- 
Ooops sorry - so the peer definitions already support registration using
the callbackextension parameter. The world is a wonderful place and I
didn't know it

The ~expiry parameter does not currently seem to be supported in the peer
definitions though. The expiry parameter is important when operating behind
nat so as to keep alive the nat binding

Anyway back to this register string issue I am happy to produce a patch
that just moves parsing of the reg->callback parameter before the call to
sip_parse_host() 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-01-26 03:21 Nick_Lewis     Note Added: 0098722                          
======================================================================