WordPress.org

Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #33821


Ignore:
Timestamp:
09/11/2015 06:32:46 AM (4 years ago)
Author:
johnbillion
Comment:

Thanks for the patch, willshouse.

This will need some unit tests.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #33821

    • Property Keywords has-patch needs-unit-tests added
    • Property Version changed from 4.3 to 2.3
    • Property Component changed from General to Canonical
    • Property Milestone changed from Awaiting Review to Future Release
  • Ticket #33821 – Description

    initial v1  
    1 In the `wp-includes/canonical.php` file the $requested_url is built starting at line 64. It combines is_ssl() for protocol, $_SERVER['HTTP_HOST'], and $_SERVER['REQUEST_URI'] - but it does not consider $_SERVER['SERVER_PORT']
     1In the `wp-includes/canonical.php` file the `$requested_url` is built starting at line 64. It combines `is_ssl()` for protocol, `$_SERVER['HTTP_HOST']`, and `$_SERVER['REQUEST_URI']` - but it does not consider `$_SERVER['SERVER_PORT']`
    22
    33This causes a redirect loop for us because we run HTTPS on port 8443.