Problem/Motivation

When using the BBCode module on a site running under SSL any YouTube embedded videos trigger an "insecure content" warning that is disruptive to the visitor flow. On some browsers this breaks the site until the visitor manually selects either a "load" or "don't load" option.

Proposed Resolution

All YouTube videos are embedded using the https protocol. This fixes the insecure content warning on sites using SSL and presents no major downsides to sites that are not using SSL. Attached patch updates the YouTube filter definition to load the https player along with using YouTube's new

embed standard.
CommentFileSizeAuthor
#1 youtubefilter-relativeprotocol.patch1.06 KBIgnigena
youtube-filter-ssl.patch1.07 KBIgnigena

Comments

Ignigena’s picture

Ignigena commented
StatusFileSize
new youtubefilter-relativeprotocol.patch1.06 KB

Updated the patch (attached)

All YouTube embeds are done with a relative protocol URL ("//www.youtube.com" vs. "http://www.youtube.com") This will automatically adopt either the HTTP or HTTPS protocol depending on which the site is running.

jordanmagnuson’s picture

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

#1 working well for me.

naudefj’s picture

naudefj commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Patch committed. Thank you so much!

Status: Fixed » Closed (fixed)

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