#37835 [Opn->Bgs]: URL rewriting uses raw value of arg_separator.output, not HTML-escaped value
ID: 37835
Updated by: mike@php.net
Reported By: ma499 at doc dot ic dot ac dot uk
-Status: Open
+Status: Bogus
Bug Type: Session related
Operating System: n/a
PHP Version: 5.1.4
New Comment:
http_build_query() in PHP-5.1+ has an optional third "arg_separator"
argument.
I suggest, if at all, this is a documentation problem.
Previous Comments:
------------------------------------------------------------------------
[2006-06-18 06:37:36] ma499 at doc dot ic dot ac dot uk
Description:
------------
Was previously submitted here and deemed to be bogus:
http://bugs.php.net/bug.php?id=30049
But I think reviwer of original report (nlopess@php.net)
misunderstood. This issue is not bogus.
As reviewer stated, PHPSESSID is appended later, so is not
translated to &. Hence, in order to produce valid HTML/
XHTML output arg_seperator.output must be set to & and
is standard practice
Up until PHP5 this has not been a problem as nothing else
uses arg_seperator.output.
BUT such a configuration now causes a problem with the new
http_build_query() in PHP5. The output of that function will
no longer be valid should the developer want to use in, for
example, a Location: header (where the & should not be
escaped).
This is therefore a bug. I would suggest the problem is with
the transid code which shoudl escape it's &PHPSESSID output
to &PHPSESSID.
Reproduce code:
---------------
See http://bugs.php.net/bug.php?id=30049
Expected result:
----------------
See http://bugs.php.net/bug.php?id=30049
Actual result:
--------------
See http://bugs.php.net/bug.php?id=30049
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=37835&edit=1
Thread (2 messages)