• Resolved B M

    (@mampf)


    Hello,

    having optional ssl, I see the thumbnails loaded over https://, even on a ssl-secured connection (https-protocol).

    I know there are some bugs in WordPress, but would it be possible to check for is_ssl() and reset the protocol? You could also use protocol-relative links like <img src="//domain.tld/path/to/image.png" >.

    I think this should be done in crp_get_the_post_thumbnail.

    https://www.ads-software.com/plugins/contextual-related-posts/

Viewing 10 replies - 1 through 10 (of 10 total)
  • Thread Starter B M

    (@mampf)

    My workaround on line 836:
    $postimage = preg_replace('~https?://~', '//', $postimage);

    Then don’t forget to empty the cache.

    Also possible:

    if (is_ssl()) {
        $postimage = preg_replace('~https://~', 'https://', $postimage);
    }

    Plugin Author Ajay

    (@ajay)

    Thanks. let me look into this.

    Wouldn’t this be a better option rather than the protocol independent link?

    if (is_ssl()) {
        $postimage = preg_replace('~https://~', 'https://', $postimage);
    }
    Thread Starter B M

    (@mampf)

    Don’t really know. I’d say, most servers are overloaded already, and let the client do as much as possible.

    On the other hand, having the protocol specified would be more consistent.

    Your choice, both goes well.

    Plugin Author Ajay

    (@ajay)

    Will run a few tests at my end. I’ve got limited access to an SSL site (not a production one unfortunately) but I can give it a go to see how it displays.

    Hi,

    do I have to fix the link to the posts the same way? All the generated links do have http instead of https (the site was switched from http to https). All other plugins do show https links. Cache was emptied. Index rebuild.

    Thanks for your help.

    Plugin Author Ajay

    (@ajay)

    Hi,

    v2.0.1 should fix the SSL issues. Could you please give this a shot?

    Thread Starter B M

    (@mampf)

    Are you sure your commit did make it? Didn’t try it yet, but I don’t see a suitable commit in the Commit log.

    Also, I won’t dare to try using a version where “wp-admin” is hardcoded. This might be a new core feature in future releases, see #13118.

    Plugin Author Ajay

    (@ajay)

    v2.0.0 has the SSL code changes. You’ll find a more detailed commit log on Github.

    v2.0.1 fixed an ajax issue.

    on wp-admin. The plugin checks for is_admin anyways and as a fall back checks wp-admin.

    I’ve noticed that the ajax clear cache button breaks at times when the wp-admin is omitted.

    Thread Starter B M

    (@mampf)

    Sorry for not replying sooner. I’ll set it to solved, works for me.

    Plugin Author Ajay

    (@ajay)

    Thank you for confirming !

Viewing 10 replies - 1 through 10 (of 10 total)
  • The topic ‘thumbnails are not loaded via https if needed’ is closed to new replies.