If your site uses HTTPS many browsers will give a mixed-content error since the FB graph pictures are using HTTP.

CommentFileSizeAuthor
#2 facebook_pull-https-1993764-2.patch1 KBrodolfovieira
#1 1993764_facebook_pull_https_1.patch2.6 KBdalin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dalin’s picture

dalin
he/him
Primary language English
Location 🇲🇽, 🇨🇦, 🌍
CreditAttribution: dalin commented
Assigned: dalin » Unassigned
Status: Active » Needs review
FileSize
1993764_facebook_pull_https_1.patch2.6 KB

The part for notes is untested, but pretty straightforward. The part for feeds works great.

rodolfovieira’s picture

rodolfovieira CreditAttribution: rodolfovieira commented
Issue summary: View changes
FileSize
facebook_pull-https-1993764-2.patch1 KB

This patch replaces 'http://' with '//' in the file 'facebook_pull-feed.tpl.php'. This will cause the browser to load remote URL using the scheme of the current site, and it will dynamically switch between http:// and https://.

Fabianx’s picture

Fabianx CreditAttribution: Fabianx commented
Status: Needs review » Reviewed & tested by the community

RTBC, looks great!

daveferrara1’s picture

daveferrara1 CreditAttribution: daveferrara1 commented
Status: Reviewed & tested by the community » Closed (fixed)

daveferrara1’s picture

daveferrara1 CreditAttribution: daveferrara1 commented
Status: Reviewed & tested by the community » Closed (fixed)