Plugin Directory


Opened 5 years ago

Closed 5 years 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


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 5 years ago.

Download all attachments as: .zip

Change History (4)

@willnorris5 years ago

comment:1 @jeherve5 years ago

  • Owner changed from mdawaffe to tmoorewp

comment:2 @georgestephanis5 years ago

  • Owner changed from tmoorewp to migueluy

comment:3 @tmoorewp5 years ago

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

In 757167:

Error: Failed to load processor CommitTicketReference
No macro or processor named 'CommitTicketReference' found
Note: See TracTickets for help on using tickets.