-
Notifications
You must be signed in to change notification settings - Fork 9.1k
Open
Labels
clarificationrequests to clarify, but not change, part of the specrequests to clarify, but not change, part of the specsecuritysecurity: authAuthentication including overlap with authorizationAuthentication including overlap with authorization
Metadata
Metadata
Assignees
Labels
clarificationrequests to clarify, but not change, part of the specrequests to clarify, but not change, part of the specsecuritysecurity: authAuthentication including overlap with authorizationAuthentication including overlap with authorization
Type
Projects
Milestone
Relationships
Development
Select code repository
Activity
jsdevel commentedon Apr 4, 2016
cc @MugeSo
jsdevel commentedon Apr 4, 2016
For example:
jsdevel commentedon Apr 4, 2016
I'm assuming this is possible because there really isn't a way to prevent this with json schema. So while it's technically possible to create this scenario according to the schema, in practice it should never happen. Is this correct?
webron commentedon Apr 4, 2016
The spec just allows you to describe things. I don't think we should go into specific cases and start saying you can describe OAuth2 together with API Key but not Basic Authentication and some other case. It complicates the spec, it complicates validation, and it's not really a concern. Just because it allows you to describe things that may not be possible doesn't mean you should do it.
jsdevel commentedon Apr 4, 2016
That's pretty much how I was thinking about it, but wanted to bring this up so others are aware. I'm working on an implementation for security and wanted to get an official clarification to before making oauth2 and basic mutually exclusive.
webron commentedon Apr 4, 2016
Right, but I don't see a need to add that to the spec itself as a restriction. Other than marking it as a potential best practices documentation plan, there's not much else to be done.
jsdevel commentedon Apr 4, 2016
Agreed! @MugeSo, based on this, I feel we can add a note in the README for
express-openapi-security
that states something to the affect that oauth2 and basic are mutually exclusive in practice but allowed by the spec.webron commentedon Apr 4, 2016
Parent: #589.