Opened 21 months ago
Last modified 21 months ago
#50044 new defect (bug)
The function get_self_link() in feed.php does not include the port
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Awaiting Review | Priority: | normal |
| Severity: | normal | Version: | 5.5 |
| Component: | Feeds | Keywords: | needs-patch |
| Focuses: | template | Cc: |
Description (last modified by )
Hi,
The function get_self_link() in /wp-includes/feed.php re-constructs the URL for the website. This is used for the atom site URL.
I have the situation where my WordPress site is running on a non-standard port. When the link for ATOM is constructed, it is missing the port.
As an example my site is running at https://wp.example.com:8080
But the get_self_link() prints it as https://wp.example.com
This might be solved by this change set: [45772]
Though in the time I spent looking it seems to be only used to generate the home root/home url of the site without any extra paths. It may or may not solve this problem correctly.
Note: See
TracTickets for help on using
tickets.
Looking at the code https://github.com/WordPress/wordpress-develop/blob/master/src/wp-includes/feed.php#L626 there is also the problem that dens't support https