[asterisk-bugs] [Asterisk 0016533]: [patch] Realtime is broken, blank strings aren't valid any more

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16533 
====================================================================== 
Reported By:                sergee
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   16533
Category:                   Addons/res_config_mysql
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
JIRA:                       SWP-616 
Regression:                 Yes 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.0 
SVN Revision (number only!): 237134 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-01-02 16:03 CST
Last Modified:              2010-01-04 12:43 CST
====================================================================== 
Summary:                    [patch] Realtime is broken, blank strings aren't
valid any more
Description: 
For a few years i've used a blank strings for values that i didn't wish to
specify. Recent commit into asterisk-addons 1.6.0 (revision 1070) broke
everything. A lot of problems arise, since my blank values aren't blank
anymore. For example, former blank fields 'fromuser' and 'fromdomain'
become not blank with this patch, and i get an error: SIP/2.0 400 Bad
Request – ‘Malformed/Missing FROM: field’.
Because my From header looks like this: From: "1000" <sip: @
>tag=as3e28b5d2

This problem doesn't have a simple solution, if you wish to support two
different blank values (empty strings and NULL) you should implement that
in the asterisk first. Replacing either value with non-empty strings is a
very bad idea, because there will be a lot of problems in both cases.

For now i wish that change would be reverted. Thank you in advance!
====================================================================== 

---------------------------------------------------------------------- 
 (0115972) svnbot (reporter) - 2010-01-04 12:43
 https://issues.asterisk.org/view.php?id=16533#c115972 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 237412

_U  branches/1.6.1/
U   branches/1.6.1/main/config.c

------------------------------------------------------------------------
r237412 | tilghman | 2010-01-04 12:43:40 -0600 (Mon, 04 Jan 2010) | 14
lines

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

........
  r237410 | tilghman | 2010-01-04 12:42:10 -0600 (Mon, 04 Jan 2010) | 7
lines
  
  Further reduce the encoded blank values back to blank in the realtime
API.
  (closes issue https://issues.asterisk.org/view.php?id=16533)
   Reported by: sergee
   Patches: 
         200100104__issue16533.diff.txt uploaded by tilghman (license 14)
   Tested by: sergee
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-01-04 12:43 svnbot         Checkin                                      
2010-01-04 12:43 svnbot         Note Added: 0115972                          
======================================================================