Option to Append https

  • Resolved luxinterior

    (@luxinterior)


    7 years ago

    I’m using your plugin on my site and would like to be able to force https to the beginning of cdn links. At the moment it’s stripping them and just putting //cdn.xxx.com but I need to have them there to comply with facebook og:image requirements. Without https the image link is flagged as invalid and sometimes it will post the image and sometimes it won’t. I’ve tried appending it via code but for some reason, it comes out as ‘https:https://cdn.xxx.com’ which is strange.

    Any help would be appreciated.

    Lux

Viewing 5 replies - 1 through 5 (of 5 total)
  • Mark Barnes

    (@mark8barnes)

    Yes, I have exactly the same problem. In my case, it’s because I use an RSS to email service, and email clients require the protocol (http or https). I know stripping the protocol is valid HTML, but there’s really no benefit in doing so when it has so many side-effects.

    Mark Barnes

    (@mark8barnes)

    Whilst we wait for BunnyCDN to fix this, it’s relatively easy to modify the code.

    On line 66 of bunnycdn.php, just replace the "//" with "https://" or "https://". Or, better, replace it with (is_ssl() ? 'https://' : 'https://')

    In the latter case, the line will now be:

    $rewriter = new BunnyCDNFilter($baseUrl, (is_ssl() ? 'https://' : 'https://').$options["cdn_domain_name"], $options["directories"], $options["excluded"]);

    Thread Starter luxinterior

    (@luxinterior)

    Thanks for the tip Mark! Hopefully, BunnyCDN will do something about it soon.

    Lux

    Plugin Author bunny.net

    (@bunnycdn)

    Hello Lux, Mark,

    Sorry about the issue and a bit of a delayed response. Thank you, Mark, for helping out in the meanwhile. We’ve now updated our plugin to correct the problem.

    BunnyCDN

    Thanks!

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Option to Append https’ is closed to new replies.