If you are using HTTPs, or a mix of HTTP and HTTPs, this module will not show the videos, because they are served over HTTP only. Even if you specify an HTTPs url for youtube, it goes to HTTP, because the module is just parsing the video ID out of the URL and discarding the rest.

The fix is to use protocol relative URLs when generating the embed address.

The attached patch fixes youtube videos, but I did not look at what other sites support serving over both HTTP and HTTPs.

(Apologies if the attached patch is in the wrong format...I'm not quite sure how to do it properly)

CommentFileSizeAuthor
#1 2438439-video-filter-protocol-fix.patch1.46 KBgrom358
video_filter-videos_over_https.diff1.63 KBgrantemsley
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grom358’s picture

grom358 CreditAttribution: grom358 commented
FileSize
2438439-video-filter-protocol-fix.patch1.46 KB

Slight fix to format of patch file supplied by @grantemsley

grom358’s picture

grom358 CreditAttribution: grom358 commented
Status: Active » Needs review
Chris Gillis’s picture

Chris Gillis CreditAttribution: Chris Gillis commented

Confirm this patch works. +1 from me. :)

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

+1

grom358’s picture

grom358 CreditAttribution: grom358 commented
Status: Needs review » Reviewed & tested by the community
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Parent issue: » #2297491: Plan for Video Filter v7.x-3.2 release

+1

blackdog’s picture

blackdog CreditAttribution: blackdog as a volunteer commented
Status: Reviewed & tested by the community » Closed (duplicate)

This was fixed in #786774.