Quantcast
Channel: WordPress.org Forums » [Jetpack - WP Security, Backup, Speed, & Growth] Support
Viewing all articles
Browse latest Browse all 37014

Will Norris on "[Plugin: Jetpack by WordPress.com] photon and https URLs"

$
0
0

Thanks Jeremy, glad to see this in! One comment though is that the name of the filter implies the opposite of what it is doing. It says "bail on https", implying that setting this to true would make it bail on https. That would be a wrong assumption though, since setting it to true actually means that it *won't* bail on https.

And technically, it's not bailing or not bailing based on whether the link is "https", it's bailing or not based on whether it is anything other than "http". In practice, that distinction is probably not important, but just pointing it out.

So maybe a filter name along the lines of "jetpack_photo_allow_https" or something similar would be clearer?

In any event, thanks for putting this in. :)


Viewing all articles
Browse latest Browse all 37014

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>