1. This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

Fixed Facebook icon causes mixed-content warning

Discussion in 'Resolved Bug Reports' started by Wootalyzer, Oct 22, 2011.

  1. Wootalyzer

    Wootalyzer Active Member

    I've been resisting posting this bug for a while, because it's almost off the charts on the scale of "not being very important." However, I figured I'd try to get it thrown in before 1.1 to help myself (and hopefully others) with some template headaches.

    Long story short, the facebook login icon currently used is http://static.ak.fbcdn.net/images/connect_sprite.png
    It should be //www.facebook.com/images/connect_sprite.png

    When running your site over HTTPS, even if you override this in EXTRA.css, some browsers will still throw a mixed content warning because they try to load the http image prior to the one overriding it. This is the only remaining mixed-content issue with XenForo, aside from [img] tags (which are a huge unrelated problem on their own, and should not IMO be handled in the XenForo core).

    Long live https :)
  2. Kier

    Kier XenForo Developer Staff Member

  3. Thomas P

    Thomas P Member

  4. Wootalyzer

    Wootalyzer Active Member

  5. Thomas P

    Thomas P Member

    Thanks Wootalyzer, our forums have heavy load being a Big Board so this is unfortunately not an option for us.

    Glad that you issue was fixed btw! :)
  6. Wootalyzer

    Wootalyzer Active Member

    Don't underestimate the power of modern processing ;). SSL support has come a long way since the origination of https, and our server's load average hovers around 0.10 all day long. While there are certainly other reasons for not switching to full HTTPS, load is quickly becoming a non-issue. Session re-use also helps by eliminating a lot of the extra handshaking latency.

    Github did a nice piece on the benifits / drawbacks of their full HTTPS conversion. It's quite the interesting read, and the comments on the articles bring up some good points.
    Thomas P likes this.

Share This Page