Opened 2 years ago

Last modified 7 months ago

#17718 new defect (bug)

Don't redirect if user is already on port 80

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

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 23 months ago.
17718.2.diff (469 bytes) - added by MikeHansenMe 7 months ago.
refreshed

Download all attachments as: .zip

Change History (5)

comment:1   dd322 years ago

  • Component changed from General to Canonical
  • Keywords has-patch added

refreshed

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