Opened 7 years ago
Last modified 8 months ago
#44766 new defect (bug)
Microsoft-IIS/6.0 problem with wp_fix_server_vars
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | major | Version: | 4.9.8 |
Component: | Bootstrap/Load | Keywords: | needs-patch needs-unit-tests |
Focuses: | Cc: |
Description
The current function assumes problems that are fixed in many IIS versions nowadays and causes more problems. Currently, the function fails to detect mod_rewrite correctly in some instances and duplicates the query_string because it falls in the third condition on load.php:75 (duplication of the query string happens at load.php:90.
Suggestion : just look in the already existing REQUEST_URI if the query string exists.
Note: See
TracTickets for help on using
tickets.
I'm not able to move this forward since I don't have access to an IIS server. I don't know if it's possible, but I think that it would be great if we could get an automated test env setup that uses IIS in order to test this.
It's also not clear exactly what needs to change since load.php has changed over the last 6 years. A PR/patch would be nice.