[asterisk-bugs] [Asterisk 0014110]: [patch] SIPAddHeader problems with escaping and quoting

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14110 
====================================================================== 
Reported By:                gork
Assigned To:                Corydon76
====================================================================== 
Project:                    Asterisk
Issue ID:                   14110
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   tweak
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           1.6.0.3-rc1 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-12-18 17:58 CST
Last Modified:              2008-12-23 11:44 CST
====================================================================== 
Summary:                    [patch] SIPAddHeader problems with escaping and
quoting
Description: 
To include semicolons in an argument to SIPAddHeader in extensions.conf
they must be escaped. SipAddHeader does not, however, unescape them when
adding the header, thus it is impossible to include semicolons in a sip
header using extensions.conf. Semicolons are a very important part of many
SIP headers that users might want to add with this application.

Additionally, if a double quote is the last character of the argument
passed to SIPAddHeader and the entire header is not contained within double
quotes, it will not be included in the header.
====================================================================== 

---------------------------------------------------------------------- 
 (0096910) gork (reporter) - 2008-12-23 11:44
 http://bugs.digium.com/view.php?id=14110#c96910 
---------------------------------------------------------------------- 
Gotcha; yes it seemed like there was some additional functionality there.
Makes sense and looks like it works much better then. I didnt look to see
that that function was already in trunk. Thanks for the help. Looks like
case closed 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-12-23 11:44 gork           Note Added: 0096910                          
======================================================================