Skip to content
This repository has been archived by the owner on Nov 13, 2022. It is now read-only.

oauth2 support #7

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

oauth2 support #7

wants to merge 8 commits into from

Conversation

mpestritto
Copy link

Bunch of changes to support oauth2. Opening up for discussion.

  • upgrade swagger-ui core code from 2.1.2 to 2.1.3
  • fixed all images assets to load
  • additional configuration variables to control index page
  • o2c template never rendered. added it as the default callback
  • supports forcing ssl

added style.scss to application.scss
Fixed images. Were not referencing correctly from css. Easier to move image location rather than edit swagger-ui core css.
Favicon renders.
@batdevis
Copy link
Owner

@mpestritto sorry for the late response, this pr is a great job.
I will merge it by the end of the week.

@mpestritto
Copy link
Author

@batdevis Thanks. I've used this tag on two projects that require oauth logins without issues.
Let me know if you have any questions.

@blainejohnson17
Copy link

@batdevis any update on getting this merged?

@drewish
Copy link

drewish commented Oct 3, 2016

looks like it needs a rebase

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

Successfully merging this pull request may close these issues.

4 participants