WordPress.org

Make WordPress Core

Opened 4 weeks ago

Last modified 2 weeks ago

#50044 new defect (bug)

The function get_self_link() in feed.php does not include the port

Reported by: damienwhaley Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: trunk
Component: Feeds Keywords: needs-patch
Focuses: template Cc:

Description (last modified by SergeyBiryukov)

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.

Change History (2)

#1 @SergeyBiryukov
4 weeks ago

  • Description modified (diff)

#2 @Mte90
2 weeks ago

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

Note: See TracTickets for help on using tickets.