Opened 8 years ago
Last modified 5 years ago
#36486 new defect (bug)
is_apache in vars.php does not always work
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | 4.4.2 |
Component: | Rewrite Rules | Keywords: | has-patch |
Focuses: | Cc: |
Description
The $is_apache variable does not always work when using an Apache module like security2_module with the SecServerSignature option. By removing the word "Apache" from the name of the server vars.php will no longer detect that the server is Apache.
function_exists('apache_get_version')!== false |
Any Apache only function will work.
See attached patch from WP 4.4.2
I discovered this caused issues with the RewriteRules, randomly WordPress would add index.php/ within the permalinks when editing the permalink options in the wp-admin.
Not sure if this should be called a bug or an enhancement, leave it up to core team to decide that.
I would not remove the first 2 checks though, it is possible that the apache_get_version is disabled in the php.ini. I would recommend adding the option to the current checks just to be 100% sure.
Patch for this ticket from WinMerge (let me know if you need a different patch format)