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.

Sign up for GitHub

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

Jump to bottom

openIdConnectUrl is required with apiKey security schema type #50

Closed
slinkydeveloper opened this issue Jul 16, 2017 · 0 comments
Closed

openIdConnectUrl is required with apiKey security schema type #50

slinkydeveloper opened this issue Jul 16, 2017 · 0 comments

Comments

@slinkydeveloper
Copy link
Contributor

slinkydeveloper commented Jul 16, 2017 •
edited

Why "openIdConnectUrl is required" is thrown when security schema type is apiKey? It seems useless with an api key security. It throw the same error with basic security type. For example:

api_key:
    type: apiKey
    name: api_key
    in: header
    openIdConnectUrl: "http://www.example.com"
slinkydeveloper added a commit to slinkydeveloper/KaiZen-OpenApi-Parser that referenced this issue Jul 20, 2017
@slinkydeveloper
Fixes RepreZen#50
0e86d06
@slinkydeveloper slinkydeveloper mentioned this issue Jul 20, 2017
Closed
andylowry added a commit that referenced this issue Sep 15, 2017
@andylowry
[#50] Partial fix
698e00d 
This PR prevents the NPE situation in the reported example, but it does
not properly handle `operationRef` values, and creates a validation
warning stating this.

The problem is that, in all of OpenAPI 3.0, this is the *one* place
where a reference string can appear outside a reference object! And
reference handling resolves ref strings in ref objects but does not have
a way of resolving other ref strings.

So this is a stop-gap until the necessary functionality can be added to
support this properly.
andylowry added a commit that referenced this issue Sep 15, 2017
@andylowry
Merge pull request #74 from RepreZen/task/50
3e4f74a 
[#50] Partial fix
andylowry added a commit that referenced this issue Sep 18, 2017
@andylowry
[#50] Fix test yaml for BigParseTest
4274550 
File now reflects renaming of `href` to `operationRef` in `Link`
objects.
andylowry added a commit that referenced this issue Sep 18, 2017
@andylowry
Merge pull request #77 from RepreZen/task/50
4af4cd8 
[#50] Fix test yaml for BigParseTest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@slinkydeveloper