error in fvm_get_protocol

  • Resolved nicksniper2

    (@nicksniper2)


    7 years ago

    Hello!
    I’m using “CDN URL” option. The combined css and js files generates ok. The trouble is that new files refers to original (not cdn) site. If I’m watching page source, I’m expecting something like “//cdn.my.site/somethere/header.css”. But instead I have only “//my.site/somethere/header.css” – CDN URL option is not working.
    The trouble is in the function fvm_get_protocol($url) in the functions.php. The variable $wp_domain is not defined than you calling this function.
    I’ve added $wp_domain = trim(str_ireplace(array(‘https://’, ‘https://’), ”, trim(site_url(), ‘/’))); – it’s works, but not a better way to fix a problem.
    Please, fix this error, please.
    You have made a great plugin! Thank you so much!

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Raul P.

    (@alignak)

    This was a bug and it has been fixed on v2.2.2
    Please try again.

    If you need to use the “defer for pagespeed insights” option, you cannot use a cdn for js and css files, because google chrome might not load them over slow connections (thank google chrome). I’m still trying to work around that, but it’s a browser limitation.

    But you can use the cdn for anything else… just make sure you use proper headers to allow Access-Control-Allow-Origin * with fonts hosted on a cdn.

    Thread Starter nicksniper2

    (@nicksniper2)

    In version v2.2.2 I have new error – the link inside the page is https://cdn.cdn.my.site New vesion make double cdn. inside link

    Plugin Author Raul P.

    (@alignak)

    and what url is on the cdn settings page?

    Thread Starter nicksniper2

    (@nicksniper2)

    https://cdn.my.site
    the address of site is https://my.site

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘error in fvm_get_protocol’ is closed to new replies.

Tags