Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix for blocked mixed content #25

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

Fix for blocked mixed content #25

wants to merge 2 commits into from

Conversation

sbreiler
Copy link

@sbreiler sbreiler commented Jan 7, 2015

On modern browsers, calling this skript with https would create errors like "blocked mixed content".

  • i removed "http:" for the external jquery Stuff, now it uses http or https as needed
  • in Function fileprepare i replaced "http//www.shelldetector.com/api/loader.html" with "about:blank"

On modern browsers, calling this skript with https would create errors like "blocked mixed content".
 - i removed "http:" for the external jquery Stuff, now it uses http or https as needed
 - in Function fileprepare i replaced "http//www.shelldetector.com/api/loader.html" with "about:blank"
@sbreiler
Copy link
Author

sbreiler commented Jan 7, 2015

We could put www.shelldetector.com/api/loader.html back if there was support for https (Function fileprepare).

For easy usage it should be stated there is a default authentification setup and what default values are set.
@DanielRuf
Copy link

i removed "http:" for the external jquery Stuff, now it uses http or https as needed

This is considered as an anti-pattern.

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

Successfully merging this pull request may close these issues.

2 participants