[asterisk-bugs] [Asterisk 0015539]: [patch] Register request line contains wrong address when domain and registrar host differ

[Asterisk Bug Tracker]

The following issue has been CLOSED 
====================================================================== 
https://issues.asterisk.org/view.php?id=15539 
====================================================================== 
Reported By:                Nick_Lewis
Assigned To:                dvossel
====================================================================== 
Project:                    Asterisk
Issue ID:                   15539
Category:                   Channels/chan_sip/Registration
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Request Review:              
Resolution:                 reopened
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-07-20 12:00 CDT
Last Modified:              2010-01-25 11:15 CST
====================================================================== 
Summary:                    [patch] Register request line contains wrong address
when domain and registrar host differ
Description: 
If sip.conf contains a register string of the form []user at domain[]@host[]
then the address in the sip register request line is the domain even though
the request is sent to the host.

For example myself at sip.me.net@registrar.me.net results in
   REGISTER sip.me.net SIP/2.0
instead of
   REGISTER registrar.me.net SIP/2.0

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

---------------------------------------------------------------------- 
 (0117148) dvossel (administrator) - 2010-01-25 11:15
 https://issues.asterisk.org/view.php?id=15539#c117148 
---------------------------------------------------------------------- 
The patch is the problem. What your patch fixes turns out to actually be
expected behavior and broke some people's registrations.  I looked through
the commit logs and that behavior was introduced as a feature.  I guess it
makes it possible to represent more complex registrations.

Perhaps bring this up on the dev-list. If I remember correctly I believe
OEJ introduced this in this commit.
http://svnview.digium.com/svn/asterisk/branches/1.4/channels/chan_sip.c?r1=126899&r2=126902

I'm closing this issue again, but feel free to re-open it again if you can
figure out a way to get your patch in and make everyone else happy as well.
 It would probably have to be a Trunk only fix unless some new ugly option
is used to turn it on and off.  This must be a fairly common usage of the
registration string because it broke boxes even within our test lab when we
upgraded. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-01-25 11:15 dvossel        Note Added: 0117148                          
2010-01-25 11:15 dvossel        Status                   assigned => closed  
======================================================================