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

fix: bug where if no securityschemes were present, we might return valid security #893

Merged
merged 1 commit into from
Aug 24, 2020

Conversation

erunion
Copy link
Member

@erunion erunion commented Aug 24, 2020

🧰 What's being changed?

This fixes a bug in our Operation.getSecurity() code where if an API definition had a valid security definition on either an operation or the root level, we'd return that even if they didn't have any securitySchemes set up.

This bug would expose itself by throwing a fatal error when attempting to make an API request.

Screen Shot 2020-08-24 at 11 52 43 AM

See readmeio/oas#259 for the bugfix in @readme/oas-tooling.

@erunion erunion added type:bug Something isn't working scope:dependency Pull requests that update a dependency file area:explorer labels Aug 24, 2020
@erunion erunion merged commit 6aba7bf into next Aug 24, 2020
@erunion erunion deleted the fix/weird-auth-issues branch August 24, 2020 20:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
scope:dependency Pull requests that update a dependency file type:bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants