[asterisk-bugs] [Asterisk 0014185]: [patch] Setting registration expiry in registration string does not work

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14185 
====================================================================== 
Reported By:                Nick_Lewis
Assigned To:                oej
====================================================================== 
Project:                    Asterisk
Issue ID:                   14185
Category:                   Channels/chan_sip/Registration
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Target Version:             1.6.2
Asterisk Version:           1.6.0 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-01-07 05:09 CST
Last Modified:              2009-01-29 05:23 CST
====================================================================== 
Summary:                    [patch] Setting registration expiry in registration
string does not work
Description: 
In some applications it is necessary for an asterisk pbx to have very short
registrations with a voip trunk provider if for example the asterisk pbx is
behind a nat firewall and the provider does not provide other udp binding
keep alive mechanisms.

According to sip.conf the format for the registration string is
[transport://]user[:secret[:authuser]]@host[:port][/contact][~expiry]
so that it should be possible to set a shorter registration by appending a
~ and an expiry value e.g. http://bugs.digium.com/view.php?id=22#c60 for a one
minute registration

However the chan_sip.c code does not seem to make use of the [~expiry]
portion so the default expiry value is always used for all registration
strings.


======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
parent of           0014321 [patch] The contact exten field in the ...
====================================================================== 

---------------------------------------------------------------------- 
 (0099037) svnbot (reporter) - 2009-01-29 05:23
 http://bugs.digium.com/view.php?id=14185#c99037 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 172235

_U  branches/1.6.1/
U   branches/1.6.1/channels/chan_sip.c

------------------------------------------------------------------------
r172235 | oej | 2009-01-29 05:23:43 -0600 (Thu, 29 Jan 2009) | 15 lines

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

........
r172234 | oej | 2009-01-29 12:19:29 +0100 (Tor, 29 Jan 2009) | 7 lines

Make sure register= line supports both port and expiry at the same time.
(closes issue http://bugs.digium.com/view.php?id=14185)
Reported by: Nick_Lewis
Patches: 
      chan_sip.c-expiryrequest6.patch uploaded by Nick (license 657)
Tested by: Nick_Lewis

........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-01-29 05:23 svnbot         Checkin                                      
2009-01-29 05:23 svnbot         Note Added: 0099037                          
======================================================================