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

Merge apiKey-as-parameter info into security #69

Open
jonthegeek opened this issue Oct 9, 2023 · 0 comments
Open

Merge apiKey-as-parameter info into security #69

jonthegeek opened this issue Oct 9, 2023 · 0 comments
Milestone

Comments

@jonthegeek
Copy link
Owner

In at least one case (https://api.apis.guru/v2/specs/fec.gov/1.0/openapi.yaml), every path has an api_key parameter, which overlaps with the apiKey security scheme (which is defined as being in the query so it overlaps conceptually, too). The path versions give a default value, which seems reasonable.

Add default values to api_key security schemes (as an optional field which isn't possible to fill directly from the APID). After building the initial APID (which must include path parameters before this will make sense), merge info from overlapping parameters into the api_key specification.

By here (if not before) as_rapid() should call a function that does "clean up" work like this + the base_url stuff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

1 participant