The javascript reference to static.addtoany.com/menu/page.js in addtoany_page_alter() checks for HTTPS on the server and changes the protocol to http or https based on that. We ran into an issue where this check was failing on our environment. This could be due to running Pound and Varnish running on that server. At any rate, a better method would probably be to use the agnostic // protocol, which will work for either http or https.

CommentFileSizeAuthor
#2 addtoany-https-2785151-01.patch545 bytesseanbfuller

Comments

seanbfuller created an issue. See original summary.

seanbfuller’s picture

Version: 7.x-4.12 » 7.x-4.x-dev
StatusFileSize
new545 bytes

A quick patch attached against 7.x-4.x

seanbfuller’s picture

Status: Active » Needs review

  • micropat committed d9a1134 on 7.x-4.x authored by seanbfuller
    Issue #2785151 by seanbfuller: SSL check for javascript referencing...
micropat’s picture

Status: Needs review » Fixed

Thanks for reporting. Protocol relative URLs have a different set of problems, so it's time to make it exclusively HTTPS just like the D8 module.

Status: Fixed » Closed (fixed)

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

seanbfuller’s picture

Ah, good call. Thanks for the correction.