WordPress.org

Ready to get started?Download WordPress

Forums

Use Google Libraries
[resolved] why custom is_ssl test? (11 posts)

  1. webaware
    Member
    Posted 1 year ago #

    G'day Jason,

    Nice plugin, I try to push it wherever I see someone trying to replace jQuery with a CDN version :)

    Q: why are you doing the is_ssl test yourself, when WordPress does this already in the is_ssl() function? What you're doing (since you fixed the port number!) isn't any better than what WordPress is doing itself.

    BTW, it still won't work behind a reverse proxy (nor will is_ssl()) but there are host-specific workarounds for that so you shouldn't join that game anyway.

    cheers,
    Ross

    http://wordpress.org/extend/plugins/use-google-libraries/

  2. Jason Penney
    Member
    Plugin Author

    Posted 1 year ago #

    When I wrote that, WordPress had no such method. Do you know when it was added?

  3. webaware
    Member
    Posted 1 year ago #

    @since 2.6.0

  4. Jason Penney
    Member
    Plugin Author

    Posted 1 year ago #

    That explains it, UGL supported WordPress 2.4 when it launched. Probably don't still need it.

  5. webaware
    Member
    Posted 1 year ago #

    Yeah, shouldn't be too many people using 2.4 any more, and I doubt any of them will have UGL (not updated anyway!)

    http://w3techs.com/technologies/details/cm-wordpress/all/all

    http://w3techs.com/technologies/details/cm-wordpress/2/all

  6. Jason Penney
    Member
    Plugin Author

    Posted 1 year ago #

    But, if you have 3.5 or greater, is_ssl isn't called, so it's a bit of a wash there. I'll clean it up next time I'm doing something else in there. Thanks for pointing it out.

  7. webaware
    Member
    Posted 1 year ago #

    Why not? (re: >= 3.5)

  8. Jason Penney
    Member
    Plugin Author

    Posted 1 year ago #

    Oh, I take that back. WP 3.5 added support for protocol relative URLs, and I added support for that to UGL, but apparently I never committed it to SVN and released that code. Whoops!

  9. webaware
    Member
    Posted 1 year ago #

    Ah. That would be nice, better than playing the is_ssl() game all the time :)

  10. Jason Penney
    Member
    Plugin Author

    Posted 1 year ago #

    Ok, 1.5.2 uses protocol relative URLS on 3.5+, otherwise: is_ssl().

  11. wolfgang1949
    Member
    Posted 1 year ago #

    Hello Jason, I just wanted to let you know that before the recent update everything seemed to be working fine, but since the update I have found that it conflicts with a plugin (Glass magnifier) that is important for my jewelry site.

    I have deactivated it for now.

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic

Tags

No tags yet.