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

change generator URL to use https so they are not blocked #1314

Closed
wants to merge 8 commits into from
Closed

change generator URL to use https so they are not blocked #1314

wants to merge 8 commits into from

Conversation

gitolicious
Copy link
Contributor

When calling a deployed Swagger Editor in i.e. Chrome via a https url, the generators are blocked because the link is http and the browser does not allow https/http mixed-content. By loading the generators via https, this problem is solved.

@webron webron requested a review from shockey May 4, 2017 15:52
@shockey shockey self-assigned this May 10, 2017
@shockey
Copy link
Contributor

shockey commented May 10, 2017

Good catch @gitolicious!

Can you merge/rebase your branch so I can merge this PR?

@gitolicious
Copy link
Contributor Author

Done. (Hope I got it right...)

@shockey
Copy link
Contributor

shockey commented May 11, 2017

Looks like something got out of whack - no worries, I'll merge the change in in another PR and attribute the commit to you.

@gitolicious
Copy link
Contributor Author

Thanks a lot!

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