-
Notifications
You must be signed in to change notification settings - Fork 81
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
Youtube scripts will be blocked on HTTPS page #43
Comments
Thank you, your solution is perfect to me. PS : What do you think about detecting current protocol automatically by default ? |
thanks for implementing this. :) unfortunately dailymotion has the same issue regarding browser warnings, but they don't have a url parameter like "scheme" for it. so their html is just broken and i need to do a search&replace (http -> https) to get my embeds working. :-/ automatic detection is a nice idea. if it works reliably and can be switched off, why not have it by default. |
I was in the process of implementing it when I remembered this simple trick: <iframe src="//www.youtube.com/..." />Using |
sorry, i can't test this right now. could you reproduce my issue? |
any solution for https ? |
I didn't have much time to work on it recently... I'll try again when I can. |
i work with this hack now but it's not beatifull : |
this indeed is a problem if you have a site under https. I am also doing a str replace like
it would be nice if you could request the scheme as an option, or it would be sufficient to use // as @felixgirault said. The problem seems to be the embed info comes directly from the oembed provider, and I don't think youtube gives you that option. Pls correct me if I am wrong. |
As @meinharrd pointed out, there is a way to get https URLs from Youtube, but not from all providers. I don't really know what to do about this, as I don't know much about https... |
Hulu has correct embed url available only in og:video tag. Oembed always return http version. So it seems that html block need to be formed on our side, using data from meta-tags. that return us to issue #110 about chaining different providers. Because author info is not available on OpenGraph |
Issue:
Solution:
Implementation:
References:
The text was updated successfully, but these errors were encountered: