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

Enable apikey security for all methods. #194

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

Conversation

zhirsch
Copy link

@zhirsch zhirsch commented Oct 1, 2017

Also explicitly specify apikey auth for methods that require oauth2,
since otherwise the local setting (oauth2 only) would override the
global setting (apikey only). The result is that these methods
require both oauth2 and apikey authentication.

Without this fix, go-swagger doesn't generate an API that allows an
authentication method to be specified.

Fixes #193

Also explicitly specify apikey auth for methods that require oauth2,
since otherwise the local setting (oauth2 only) would override the
global setting (apikey only).  The result is that these methods
require both oauth2 and apikey authentication.

Without this fix, go-swagger doesn't generate an API that allows an
authentication method to be specified.
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.

OpenAPI 2.0 spec doesn't require apiKey auth for methods
1 participant