Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#19563 closed defect (bug) (fixed)

feed generator does not correctly detect if we run on https

Reported by: duritong Owned by: dd32
Milestone: 3.4 Priority: normal
Severity: normal Version:
Component: Feeds Keywords: has-patch
Focuses: Cc:


wp-includes/feed.php does not correctly detect, whether the feed is accessed via https or not, so it will always generate an http-only link.

This is because it looks for


instead of


Other wordpress scripts are already looking for:


So fix is quite simple and patch is attached and uploaded here: https://gist.github.com/1481164

Attachments (1)

fix.patch (553 bytes) - added by duritong 4 years ago.
fix for https on feed issue

Download all attachments as: .zip

Change History (5)

4 years ago

fix for https on feed issue

#1 @kawauso
4 years ago

  • Keywords has-patch added; rss https removed

#2 @johnbillion
4 years ago

It should actually be using is_ssl().

#3 @dd32
4 years ago

  • Milestone changed from Awaiting Review to 3.4
  • Owner set to dd32
  • Status changed from new to assigned

Yep, we should be using is_ssl() here.

#4 @dd32
4 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In [19598]:

Use is_ssl() in self_link(). Fixes #19563

Note: See TracTickets for help on using tickets.