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

Http/Https Independent Requests #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BigDub
Copy link

@BigDub BigDub commented Jan 6, 2014

Removed "http:" from service urls. This allows the requests to match whatever protocol the page was loaded in. If the user was in https the video requests will be https. If the user was in http, the requests will be http. Admittedly, I only tested 'youtube' because that is what I needed. It should work for the others but, again, I've not tested it.

whatever protocol the page was loaded in. If the user was in https the
video requests will be https. If the user was in http, the requests will
be http. Admittedly, I only tested generic 'youtube' one because that is
what I needed. It should work for the others, but I've not tested it.
@TomMettam
Copy link

Most of these services don't support https, so you'll break support on the older browsers. It needs to be done on a service-by-service basis, i think.

@ttguy
Copy link

ttguy commented Aug 13, 2014

I just want to vote that this fix should be improved and implemented - make it youtube specific and deploy it. There are situations where the EmbedVideo extension fails because of secure wiki pages and insecure youtube calls. Recently userbase.kde.org changed the wiki to make SSL manditory accross the site. And this broke youtube embeds in modern browsers because the page was delivering mixed content and the browser blocked the insecure YT content - leaving a blank space on the page. So you should allow prasad83s fix or something like it to fix this issue. userbase.kde.org hacked a fix into EmbedVideo.Services.php to get around this issue but it should work out of the box.

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.

3 participants