php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #76719 PECL REST channel returns invalid XML
Submitted: 2018-08-08 02:24 UTC Modified: 2020-06-10 15:02 UTC
Votes:4
Avg. Score:4.8 ± 0.4
Reproduced:3 of 3 (100.0%)
Same Version:2 (66.7%)
Same OS:2 (66.7%)
From: dzuelke at gmail dot com Assigned: cmb (profile)
Status: Duplicate Package: PECL website (PECL)
PHP Version: Irrelevant OS: irrelevant
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem:
40 + 12 = ?
Subscribe to this entry?

 
 [2018-08-08 02:24 UTC] dzuelke at gmail dot com
Description:
------------
Same again as #58446, #59098, #59666...

Several endpoints in the API return invalid XML, the issue is always the same: weird XML prolog and extra elements in the middle of the document; sometimes just one more time, sometimes more, e.g. one in Authentication, three in Caching:

...
<pi>
<p><n>PAM</n>
 <c>pecl.php.net</c>
 <ca xlink:href="/rest/c/Authentication">Authentication</ca>
 <l>PHP</l>
 <s>PAM integration</s>
 <d>This extension provides PAM (Pluggable Authentication Modules) integration. PAM is a system of libraries that handle the authentication tasks of applications and services. The library provides a stable API for applications to defer to for authentication tasks.</d>
 <r xlink:href="/rest/r/pam"/>
</p><?xml version="1.0" encoding="UTF-8" ?>
<a xmlns="http://pear.php.net/dtd/rest.allreleases"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xlink="http://www.w3.org/1999/xlink"     xsi:schemaLocation="http://pear.php.net/dtd/rest.allreleases
    http://pear.php.net/dtd/rest.allreleases.xsd">
 <p>pam</p>
 <c>pecl.php.net</c>
 <r><v>1.0.3</v><s>stable</s></r>
 <r><v>1.0.2</v><s>stable</s></r>
 <r><v>1.0.1</v><s>stable</s></r>
 <r><v>1.0.0</v><s>stable</s></r>
</a>

The test script below outputs the number of XML prologs for each category that has more than one in the XML.

Test script:
---------------
for f in $(curl https://pecl.php.net/rest/c/categories.xml | grep -o 'c xlink:href="[^"]*' | cut -d'"' -f2 | sed 's/info.xml/packagesinfo.xml/' | sed s#^#https://pecl.php.net#); do { echo -n "$f: "; curl --silent $f | grep '<?xml' | wc -l; } | grep -v 1; done

Actual result:
--------------
https://pecl.php.net/rest/c/Authentication/packagesinfo.xml:        2
https://pecl.php.net/rest/c/Caching/packagesinfo.xml:        4
https://pecl.php.net/rest/c/Database/packagesinfo.xml:        2
https://pecl.php.net/rest/c/File%2BFormats/packagesinfo.xml:        2
https://pecl.php.net/rest/c/GUI/packagesinfo.xml:        2
https://pecl.php.net/rest/c/Images/packagesinfo.xml:        3
https://pecl.php.net/rest/c/Networking/packagesinfo.xml:        3
https://pecl.php.net/rest/c/Numbers/packagesinfo.xml:        2
https://pecl.php.net/rest/c/PHP/packagesinfo.xml:        2
https://pecl.php.net/rest/c/System/packagesinfo.xml:        2
https://pecl.php.net/rest/c/Text/packagesinfo.xml:        2
https://pecl.php.net/rest/c/Web%2BServices/packagesinfo.xml:        2

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2018-08-08 17:35 UTC] cmb@php.net
It seems to me that this code[1] didn't work as hoped.

[1] <https://github.com/php/web-pecl/blob/da4fe2a/include/pear-rest.php#L133-L139>
 [2018-08-08 21:22 UTC] dzuelke at gmail dot com
Ugh, XML assembled via string concatenation? No wonder this is buggy.
 [2020-06-10 15:02 UTC] cmb@php.net
-Status: Open +Status: Duplicate -Assigned To: +Assigned To: cmb
 [2020-06-10 15:02 UTC] cmb@php.net
Since bug #59098 is still open, I'm closing this as duplicate.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Fri Mar 29 13:01:29 2024 UTC