Line 20: Instead of http://, should be changed to https:// to avoid Non-Secure warnings on SSL sites that are using the BU module. Our site (www.dentistry.ucla.edu) is using SSL by default and when I first installed the module, IE 8 users on Windows XP would encounter the "Do you want to view only the webpage content that was delivered securely?" message. If they clicked YES, the Browser Update bar would not show.

I have changed it to https:// and verified that it works fine on SSL sites. I imagine that it would work fine on non-SSL sites as well but have not tested this.

CommentFileSizeAuthor
bu_js.zip461 bytesdgalley
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pebosi’s picture

Status: Needs review » Fixed

As mentioned in the blog on http://browser-update.org/blog.html, they now have ssl certificates installed. So i updated the code.

  • pebosi committed 76af587 on 7.x-1.x
    Issue #2236117, #2083247 patch by sitiveni: Use protocol relative url
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.