Get WordPress

Making WordPress.org

Opened 8 years ago

Closed 8 years ago

#2037 closed defect (bug) (fixed)

WordPress.tv: Mixed content warning for Typekit JS

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: coffee2code's profile coffee2code
Milestone: Priority: normal
Component: WordPress.tv Keywords: has-patch commit
Cc:

Description

Reported by @targz-1 in ​https://wordpress.slack.com/archives/wptv/p1473935926000222:

Heya, just a comment about the wordcamp.tv website : in Chrome 52/OSX, I got a warning that the site is loading an insecure script. Indeed, in the page header, the link to the typekit js file is going over http:// – should be https:// or just //.

Attachments (2)

meta-2037.patch​ (743 bytes) - added by SergeyBiryukov 8 years ago.
meta-2037.2.patch​ (1.4 KB) - added by SergeyBiryukov 8 years ago.

Download all attachments as: .zip

Change History (6)

@SergeyBiryukov
8 years ago

#1 @coffee2code
8 years ago

  • Owner set to coffee2code
  • Resolution set to fixed
  • Status changed from new to closed

In 4071:

WordPress.tv: Fix mixed content warning for Typekit JS.

Props SergeyBiryukov, targz-1.
Fixes #2037.

#2 @ocean90
8 years ago

Protocol-relative URLs are an anti-pattern. Since the script is available on https://, https:// should always be used.

@SergeyBiryukov
8 years ago

#3 @SergeyBiryukov
8 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

meta-2037.2.patch​ changes the Typekit and RSS feed URLs to use https://.

#4 @coffee2code
8 years ago

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

In 4075:

WordPress.tv: Explicitly use HTTPS for site links in page head.

Props SergeyBiryukov.
Fixes #2037.

Note: See TracTickets for help on using tickets.