WordPress.org

Make WordPress Core

Opened 20 months ago

Last modified 18 months ago

#36486 new defect (bug)

is_apache in vars.php does not always work

Reported by: amandato Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 4.4.2
Component: Rewrite Rules Keywords:
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.

A simple solution is to add an addition check:
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.

Attachments (1)

vars.php.patch (339 bytes) - added by amandato 20 months ago.
Patch for this ticket from WinMerge (let me know if you need a different patch format)

Download all attachments as: .zip

Change History (2)

@amandato
20 months ago

Patch for this ticket from WinMerge (let me know if you need a different patch format)

#1 @amandato
18 months ago

Did I place this ticket in the right component?

Note: See TracTickets for help on using tickets.