Context Navigation

Changes between Initial Version and Version 1 of Ticket #33821

Timestamp:

09/11/2015 06:32:46 AM (10 years ago)

Author:

johnbillion

Comment:

Thanks for the patch, willshouse.

This will need some unit tests.

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

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']~~
	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']`
2	2
3	3	This causes a redirect loop for us because we run HTTPS on port 8443.