Ticket #20 (closed bug: fixed)

Opened 11 months ago

Last modified 11 months ago

Wrap error messages in valid xml when using new RPC API

Reported by: nhnFreespirit@… Owned by: vollmerk
Priority: major Milestone:
Component: Other Version: 3.4.x
Keywords: Cc:

Description (last modified by vollmerk) (diff)

When using the new API to browse content from within Amarok, error messages are somtimes mixed in with the reply XML. This causes XML parser to fail. Instead, please wrap these errors in valid XML. Here is an example repply as seen by Amarok2:

amarok:  Authentication reply:  "<br />
<b>Warning</b>:  error_log(/home/ampache/log/ampache.20071217.log) [<a href='function.error-log'>function.error-log</a>]: failed to open stream: Permission denied in <b>/home/ampache/public_html/lib/log.lib.php</b> on line <b>38</b><br />
Error: Unable to write to log (/home/ampache/log/ampache.20071217.log) Please check your log_path variable in ampache.cfg.php<br />
<b>Warning</b>:  error_log(/home/ampache/log/ampache.20071217.log) [<a href='function.error-log'>function.error-log</a>]: failed to open stream: Permission denied in <b>/home/ampache/public_html/lib/log.lib.php</b> on line <b>38</b><br />
Error: Unable to write to log (/home/ampache/log/ampache.20071217.log) Please check your log_path variable in ampache.cfg.php<br />
<b>Warning</b>:  error_log(/home/ampache/log/ampache.20071217.log) [<a href='function.error-log'>function.error-log</a>]: failed to open stream: Permission denied in <b>/home/ampache/public_html/lib/log.lib.php</b> on line <b>38</b><br />
Error: Unable to write to log (/home/ampache/log/ampache.20071217.log) Please check your log_path variable in ampache.cfg.phpError: Unable to write to log (/home/ampache/log/ampache.20071217.log) Please check your log_path variable in ampache.cfg.php<br />
<b>Warning</b>:  error_log(/home/ampache/log/ampache.20071217.log) [<a href='function.error-log'>function.error-log</a>]: failed to open stream: Permission denied in <b>/home/ampache/public_html/lib/log.lib.php</b> on line <b>38</b><br />
Error: Unable to write to log (/home/ampache/log/ampache.20071217.log) Please check your log_path variable in ampache.cfg.php<br />
<b>Warning</b>:  error_log(/home/ampache/log/ampache.20071217.log) [<a href='function.error-log'>function.error-log</a>]: failed to open stream: Permission denied in <b>/home/ampache/public_html/lib/log.lib.php</b> on line <b>38</b><br />
Error: Unable to write to log (/home/ampache/log/ampache.20071217.log) Please check your log_path variable in ampache.cfg.php<br />
<b>Warning</b>:  error_log(/home/ampache/log/ampache.20071217.log) [<a href='function.error-log'>function.error-log</a>]: failed to open stream: Permission denied in <b>/home/ampache/public_html/lib/log.lib.php</b> on line <b>38</b><br />
Error: Unable to write to log (/home/ampache/log/ampache.20071217.log) Please check your log_path variable in ampache.cfg.phpError: Unable to write to log (/home/ampache/log/ampache.20071217.log) Please check your log_path variable in ampache.cfg.php<?xml version="1.0" encoding="utf-8" ?>
<root>
        <auth><![CDATA[4e9cd0b8865e47395f1bbfa1fc4679d8]]></auth>
        <api><![CDATA[340001]]></api>
 
</root>

- Nikolaj

Attachments

Change History

Changed 11 months ago by vollmerk

  • status changed from new to closed
  • resolution set to fixed
  • description modified (diff)

fixed in r1357

Add/Change #20 (Wrap error messages in valid xml when using new RPC API)

Author



Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.