WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#19563 closed defect (bug) (fixed)

feed generator does not correctly detect if we run on https

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

Description

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

$_SERVER['https']

instead of

$_SERVER['HTTPS']

Other wordpress scripts are already looking for:

$_SERVER['HTTPS']

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 13 years ago.
fix for https on feed issue

Download all attachments as: .zip

Change History (5)

@duritong
13 years ago

fix for https on feed issue

#1 @kawauso
13 years ago

  • Keywords has-patch added; rss https removed

#2 @johnbillion
13 years ago

It should actually be using is_ssl().

#3 @dd32
13 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
13 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.