[asterisk-bugs] [Asterisk 0018186]: [patch] Malformed XML response

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18186 
====================================================================== 
Reported By:                nivaldomjunior
Assigned To:                twilson
====================================================================== 
Project:                    Asterisk
Issue ID:                   18186
Category:                   Core/ManagerInterface
Reproducibility:            always
Severity:                   trivial
Priority:                   normal
Status:                     closed
Target Version:             1.6.2.19
Asterisk Version:           SVN 
JIRA:                       SWP-2487 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): 1.8 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-10-22 07:53 CDT
Last Modified:              2011-03-10 10:09 CST
====================================================================== 
Summary:                    [patch] Malformed XML response
Description: 
Using the AJAM and XML to access the manager interface we receive a
malformed XML response.
We tested using a java and a php application, and both had received the
ajax response with error. The final tag </ajax-response> is not there.

We tested using the browser and the results are:

Chrome - Didn't worked. No </ajax-response> tag:
<ajax-response> 
<response type='object' id='unknown'><generic response='Success'
message='Authentication accepted' /></response> 

Safari - Didn't worked. The </ajax-response> tag is there, but is not
complete. The > is not there:
<ajax-response>
<response type='object' id='unknown'><generic response='Success'
message='Authentication accepted' /></response>
</ajax-response

Firefox - Worked. The XML response is complete:

?
<ajax-response>
?
<response type="object" id="unknown">
<generic response="Success" message="Queue summary will follow"/>
</response>
?
<response type="object" id="unknown">
<generic event="QueueSummary" queue="Fila-Unimed" loggedin="0"
available="0" callers="0" holdtime="0" talktime="0" longestholdtime="0"/>
</response>
?
<response type="object" id="unknown">
<generic event="QueueSummary" queue="fila-unimed-nne" loggedin="0"
available="0" callers="0" holdtime="0" talktime="0" longestholdtime="0"/>
</response>
?
<response type="object" id="unknown">
<generic event="QueueSummaryComplete"/>
</response>
</ajax-response>


======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
has duplicate       0018256 MXML &quot;ajax-response&quot; is not w...
====================================================================== 

---------------------------------------------------------------------- 
 (0132819) svnbot (reporter) - 2011-03-10 10:09
 https://issues.asterisk.org/view.php?id=18186#c132819 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 310241

_U  trunk/
U   trunk/main/manager.c
U   trunk/res/res_phoneprov.c

------------------------------------------------------------------------
r310241 | twilson | 2011-03-10 10:09:10 -0600 (Thu, 10 Mar 2011) | 20
lines

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

........
  r310240 | twilson | 2011-03-10 10:05:45 -0600 (Thu, 10 Mar 2011) | 13
lines
  
  Add \r\n to remaining http headers passed to ast_http_send
  
  r309204 changed the behavior of ast_http_send. It now requires headers
  to be passed with trailing \r\n. This change updates the remaining
  instances in the code that did not pass the \r\n.
  
  (closes issue https://issues.asterisk.org/view.php?id=18186)
  Reported by: nivaldomjunior
  Patches: 
        res_phoneprov.c.diff uploaded by lathama (license 1028)
        manager.diff.txt uploaded by twilson (license 396)
  Tested by: lathama
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-03-10 10:09 svnbot         Checkin                                      
2011-03-10 10:09 svnbot         Note Added: 0132819                          
======================================================================