Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] #156 Browser complains about mixed content #159

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

darylteo
Copy link

Original Issue

If a site is loaded using https:// youtube videos (and probably
other embeds) will not load because the script hard codes http
in the video URLs


The following fix is as follows:

  • only applies to video urls (any impact on other embeds unknown)
  • checks for the current document protocol (http: or https:)
  • if protocol is file:, then it defaults to http: as it is existing behaviour

See this fix in action
here
and by clicking the vimeo videos on the right sidebar

If a site is loaded using https:// youtube videos (and probably
other embeds) will not load because the script hard codes http
in the video URLs

The following fix is as follows:

 - only applies to video urls (any impact on other embeds unknown)
 - checks for the current document protocol (http: or https:)
 - if protocol is file:, then it defaults to http: as is
   existing behaviour

See this fix in action at by clicking the vimeo videos on the
right.

https://www.listcorp.com/asx/ler
@darylteo
Copy link
Author

This PR now supports the allowfullscreen property.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant