WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 2 years ago

#41877 new enhancement

Caddy server support

Reported by: viliamkopecky Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 4.8.1
Component: Permalinks Keywords: has-patch needs-refresh
Focuses: Cc:
PR Number:

Description

Caddy behaves mostly as nginx. But unknown $_SERVER['SERVER_SOFTWARE'] caused default permalinks options to include index.php. This patch replicates global $is_nginx variable to Caddy's context.

Attachments (1)

41877.patch (2.6 KB) - added by viliamkopecky 2 years ago.
adding and using of $is_caddy global variable

Download all attachments as: .zip

Change History (4)

@viliamkopecky
2 years ago

adding and using of $is_caddy global variable

#1 @JPry
2 years ago

  • Keywords has-patch added

#2 follow-up: @dd32
2 years ago

I wonder if it's about the time that we can assume that all web servers have rewrites available?

We could keep the iis7/apache checks as they expose methods to check.. but most other server environments DO support it today - and any future server is likely to support it too given how relied upon it is today.

#3 in reply to: ↑ 2 @johnbillion
2 years ago

  • Focuses administration removed
  • Keywords needs-refresh added

Replying to dd32:

I wonder if it's about the time that we can assume that all web servers have rewrites available?

We could keep the iis7/apache checks as they expose methods to check.

This sounds sensible.

Note: See TracTickets for help on using tickets.