[asterisk-bugs] [Asterisk 0018344]: regression improper sip parse when invite contains values to the left of the @ ; phone-context=+1; npdi=yes

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18344 
====================================================================== 
Reported By:                danimal
Assigned To:                jrose
====================================================================== 
Project:                    Asterisk
Issue ID:                   18344
Category:                   Channels/chan_sip/Interoperability
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           1.8.0 
JIRA:                       SWP-2637 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-11-21 13:58 CST
Last Modified:              2011-05-20 08:42 CDT
====================================================================== 
Summary:                    regression improper sip parse when invite contains
values to the left of the @ ;phone-context=+1;npdi=yes
Description: 
This is related to a circa 2006 issue
https://issues.asterisk.org/view.php?id=7761

when a call origionates from the pstn to a sonus and terminates to
asterisk 1.8 sonus adds values to the left side of the @ side in the
invite. Previous versions of asterisk would return only the telephone
number in ${EXTEN}. 1.8 is returning everything to the left of the @ sign
in ${EXTEN}/

steps to reproduce.
1. install centos 5.5
2. install asterisk 1.8 via the digium repository / yum install
3. my upstream provider uses a sonus GW for pstn terminaltion.
4. call from the pstn to asterisk 1.8
5. exten =>_X!,1,NOOP(dialed number => ${EXTEN})
6, notice the value of ${EXTEN}

anticipated results
asterisk will return only the DID number in ${EXTEN}

actual results
asterisk is returning everything to the left side of the @ symbol in
${EXTEN} this value is then also stored in the cdr records as well.
====================================================================== 

---------------------------------------------------------------------- 
 (0135185) svnbot (reporter) - 2011-05-20 08:42
 https://issues.asterisk.org/view.php?id=18344#c135185 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 319939

_U  trunk/
U   trunk/channels/chan_sip.c
U   trunk/channels/sip/include/sip.h
U   trunk/configs/sip.conf.sample

------------------------------------------------------------------------
r319939 | jrose | 2011-05-20 08:42:16 -0500 (Fri, 20 May 2011) | 19 lines

Merged revisions 319938 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r319938 | jrose | 2011-05-20 08:28:24 -0500 (Fri, 20 May 2011) | 12
lines
  
  Adds legacy_useroption_parsing to address interoperability concerns.
  
  With the new option engaged, Asterisk should interpret user fields with
useroptions
  contained within the userfield of the uri by stripping them out of the
original message
  whenever a semicolon is encountered in the userfield string.
  
  (closes issue https://issues.asterisk.org/view.php?id=18344)
  Reported by: danimal
  Tested by: jrose
  
  Review: https://reviewboard.asterisk.org/r/1223/
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-05-20 08:42 svnbot         Checkin                                      
2011-05-20 08:42 svnbot         Note Added: 0135185                          
======================================================================