WordPress.org

Making WordPress.org

Opened 19 months ago

Last modified 9 months ago

#2348 assigned enhancement

Add Filter to fix mixed content warnings on WordCamp Sites

Reported by: BandonRandon Owned by:
Milestone: Priority: low
Component: WordCamp Site & Plugins Keywords: needs-patch good-first-bug
Cc:

Description

Now that all WordCamp sites were switched to force SSL with certificates from Lets Encrypt some sites are showing mixed content warnings.

These warnings are more likely to happen on older WordCamp sites that were created before switching everything over to SSL. For example:
https://2014.seattle.wordcamp.org/

Instead of asking organizers to go back and fix these sites, we could instead add filters of either the_content or the site's buffer to force all content to be loaded from https URLs.

Potential downside would be linked images from http sources which don't support https. If a site has Jetpack, Photon should be able to rectify these situations.

Attachments (1)

Screenshot_29-12-2016-17:22:10.png (41.5 KB) - added by BandonRandon 19 months ago.
Mixed Content warning of 2014.seattle.wordcamp.org in Brave

Download all attachments as: .zip

Change History (7)

@BandonRandon
19 months ago

Mixed Content warning of 2014.seattle.wordcamp.org in Brave

#1 @iandunn
19 months ago

  • Keywords good-first-bug added
  • Owner set to iandunn
  • Status changed from new to accepted

+1

linked images from http sources which don't support https

I think we should consider only modifying links that we know support HTTPS, to avoid breaking content. Mixed content warnings are bad, not not as bad as broken images.

#2 follow-up: @dd32
19 months ago

I think we should consider only modifying links that we know support HTTPS, to avoid breaking content.

The other option would be to proxy all other http-image assets through Jetpack Photon to utilise it's SSL.

#3 in reply to: ↑ 2 @iandunn
19 months ago

Replying to dd32:

The other option would be to proxy all other http-image assets through Jetpack Photon to utilise it's SSL.

Ah, +1. I bet that's what Brooke meant in the ticket description, and I just misunderstood it.

#5 @iandunn
9 months ago

  • Owner iandunn deleted
  • Status changed from accepted to assigned

#6 @iandunn
9 months ago

  • Priority changed from normal to low
Note: See TracTickets for help on using tickets.