Making WordPress.org

Opened 5 years ago

Closed 5 years ago

#2037 closed defect (fixed)

WordPress.tv: Mixed content warning for Typekit JS

Reported by: SergeyBiryukov Owned by: coffee2code
Milestone: Priority: normal
Component: WordPress.tv Keywords: has-patch commit


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 5 years ago.
meta-2037.2.patch (1.4 KB) - added by SergeyBiryukov 5 years ago.

Download all attachments as: .zip

Change History (6)

#1 @coffee2code
5 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
5 years ago

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

#3 @SergeyBiryukov
5 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
5 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.