[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