WordPress.org

Make WordPress Core

Opened 3 years ago

Last modified 18 months ago

#17718 new defect (bug)

Don't redirect if user is already on port 80

Reported by: davidcx Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.1.3
Component: Canonical Keywords: has-patch
Focuses: Cc:

Description

If user is already using port 80 but $original['port'] is set to 80 anyway, do not send a needless redirect.

wp-includes/canonical.php line 324

change from

if ( !empty( $original['port'] ) )

change to

if ( !empty( $original['port'] ) && $original['port'] != 80 )

Fixes, for example, Nagios's check_http getting sent a redirect rather than the page content.

-davidc

Attachments (2)

17718.diff (474 bytes) - added by kawauso 3 years ago.
17718.2.diff (469 bytes) - added by MikeHansenMe 18 months ago.
refreshed

Download all attachments as: .zip

Change History (5)

comment:1 dd323 years ago

  • Component changed from General to Canonical

comment:2 kawauso3 years ago

  • Keywords has-patch added

kawauso3 years ago

MikeHansenMe18 months ago

refreshed

comment:3 MikeHansenMe18 months ago

  • Cc mdhansen@… added
Note: See TracTickets for help on using tickets.