WordPress.org

Plugin Directory

jetpack

Opened 14 months ago

Closed 8 months ago

#1653 closed enhancement (fixed)

add ability to disable jetpack's https check for photon

Reported by: willnorris Owned by: migueluy
Priority: normal Severity: normal
Plugin: jetpack Keywords: photon
Cc:

Description

Today, jetpack's photon feature will not work for image URLs that have an 'https' URL scheme. This is presumably because photon URLs don't specify the URL scheme for the original image, so defaults to using 'http'. However, in the case where the http URL redirects to the equivalent https URL, photon (the service) works just fine. to wit:

original photo: http://willnorris.com/content/uploads/2012/12/pocket-share-to-google.png (this will redirect to the same URL with https)

photon photo: https://i0.wp.com/willnorris.com/content/uploads/2012/12/pocket-share-to-google.png?resize=100,100

The jetpack plugin, however, won't even attempt to replace https image URLs with the photon equivalent, since it has no way of knowing whether the necessary http->https redirect is in place. The provided patch here adds a new filter which allows the strict https checking to be disabled for those site owners who do have the necessary redirect in place. The default behavior of skipping https URLs remains intact.

original support discussion: http://wordpress.org/support/topic/photon-and-https-urls

Attachments (1)

photon.diff (658 bytes) - added by willnorris 14 months ago.

Download all attachments as: .zip

Change History (4)

willnorris14 months ago

comment:1 jeherve13 months ago

  • Owner changed from mdawaffe to tmoorewp

comment:2 georgestephanis11 months ago

  • Owner changed from tmoorewp to migueluy

comment:3 tmoorewp8 months ago

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

In 757167:

Add a filter that allows site admins/owners to enable Photon for HTTPS URLs.

Fixes #1653

Props @willnorris

Note: See TracTickets for help on using tickets.