[wp-trac] Re: [WordPress Trac] #3727: WP->parse_request() won't
replace $pathinfo when $req_uri contains any %## encoding character.
WordPress Trac
wp-trac at lists.automattic.com
Sun Jun 3 08:05:36 GMT 2007
#3727: WP->parse_request() won't replace $pathinfo when $req_uri contains any %##
encoding character.
-----------------------------------------+----------------------------------
Reporter: Kirin_Lin | Owner: ryan
Type: defect | Status: reopened
Priority: normal | Milestone: 2.2
Component: General | Version: 2.2
Severity: critical | Resolution:
Keywords: rewrite permalink has-patch |
-----------------------------------------+----------------------------------
Comment (by hakre):
In a deeper analysis I realized that Kirin_Lin simply "changed something
that affected something the way she/he wanted" and was fine was that but
without even thinking about where the defect is originated in. This leads
to an enourmous space for side-effects and esoteric behaviour of the
rewrite part. Rawurldecode should only be applied on the query-info part
of a querystring. so this has to be done on another place in the code-
logic not in the part where decoded url-information has to stay inside the
URL as within that line of code.
You can compare the applied fix as if the apache developers would pass a
general rawurldecode() over the complete request URI.
--
Ticket URL: <http://trac.wordpress.org/ticket/3727#comment:18>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list