based on example: http://www.addtoany.com/buttons/examples/https_ssl/
I had to change addtoany.module, function _addtoany_create_button() [part of it]
to:

// default: ssl warning "this page contains both secure and not secure... " comment line
//$_addtoany_script .= '

';

//revised to:
$_addtoany_script .= '

CommentFileSizeAuthor
#4 drupal_addtoany_https_fix_497458_v1.patch953 bytesao2
#2 addtoany-5x23-httpsfix.patch626 bytesChad_Dupuis

Comments

vpapadim’s picture

vpapadim commented

sorry, but i did not format the code correctly... here is the above change again.

// default: ssl warning "this page contains both secure and not secure
//$_addtoany_script .= '</script><script type="text/javascript" src="http://static.addtoany.com/menu/page.js"></script>';

// revised to:
$_addtoany_script .= '</script><script type="text/javascript" src="https://http-s1.simplecdn.net/static.addtoany.com/menu/page.js"></script>';
Chad_Dupuis’s picture

Chad_Dupuis commented
Status: Active » Needs review
StatusFileSize
new addtoany-5x23-httpsfix.patch626 bytes

Thanks for this - I've created a patch for the change.... Although this should probably be written as a user defined toggle within the setup pages.

haus’s picture

haus commented

Hallo, I am also interested that there is a dynamic adjustment for wheather I use https oder http.
Thanks

ao2’s picture

ao2 commented
Version: 5.x-2.3 » 6.x-2.4
StatusFileSize
new drupal_addtoany_https_fix_497458_v1.patch953 bytes

@haus, I've seen code checking for $_SERVER['HTTPS'] to decide about https.

Patch attached for 6.x-2.4

Regards,
Antonio

micropat’s picture

micropat commented
Assigned: Unassigned » micropat
Status: Needs review » Patch (to be ported)

Shooting for next release.

micropat’s picture

micropat commented
Status: Patch (to be ported) » Fixed

Committed, and should make the next release.

Antonio, your patch only needed the 's' in https://addtoany.simplecdn.net/menu/page.js ;-)

ao2’s picture

ao2 commented

@micropat, yeah, sorry for the typo.

I tested the changes using the old address https://http-s1.simplecdn.net/static.addtoany.com/menu/page.js, and then failed to copy and paste the new one before submitting the final patch. :)

All the best,
Antonio

Status: Fixed » Closed (fixed)
Issue tags: -warning, -ssl, -addtoany, -https

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

Issue tags: +warning, +ssl, +addtoany, +https

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.