[wp-trac] Re: [WordPress Trac] #5160: REQUEST_URI fix for IIS misses certain configurations

[WordPress Trac]

#5160: REQUEST_URI fix for IIS misses certain configurations
-------------------------------------------------+--------------------------
 Reporter:  brh                                  |        Owner:  anonymous
     Type:  defect                               |       Status:  reopened 
 Priority:  normal                               |    Milestone:  2.4      
Component:  General                              |      Version:           
 Severity:  normal                               |   Resolution:           
 Keywords:  REQUEST_URI has-patch needs-testing  |  
-------------------------------------------------+--------------------------
Comment (by snakefoot):

 The combination of just SCRIPT_NAME + PATH_INFO seems to cover your
 configuration, without needing the ORIG_PATH_INFO as PATH_INFO is being
 set.

 Not sure if all PHP + CGI configurations configures the ORIG_PATH_INFO as
 being the combination of SCRIPT_NAME + PATH_INFO. But if they did, then
 the patch to handle the situation of no PATH_INFO would look something
 like this:
 {{{
     if (isset(ORIG_PATH_INFO) && empty(PATH_INFO) &&
 SCRIPT_NAME!=ORIG_PATH_INFO)
         REQUEST_URI = ORIG_PATH_INFO;
     else
     if (SCRIPT_NAME==PATH_INFO)
         REQUEST_URI = SCRIPT_NAME;
     else
         REQUEST_URI = SCRIPT_NAME . PATH_INFO;

     REQUEST_URI .= QUERY_INFO
 }}}

 Not sure it is a good idea to try fix a bug which cannot be tested if
 fixed. The two patches I have attached for trunk and branch will solve the
 problem for brh.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/5160#comment:10>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software