Skip to content

Conversation

@wsalembi
Copy link
Contributor

@wsalembi wsalembi commented Jan 22, 2020

Because in the v1 library operation.getSecurity() doesn't return a list of SecurityRequirements cfr the top-level security field in the swagger file, I opted for adding an empty Map to the list to represent the none security requirement.

We need swagger-parser to retain the unsecured security requirement on an operation level. Now it is stripped

@gracekarina gracekarina merged commit c3949ab into swagger-api:v1 Jan 31, 2020
@gracekarina
Copy link
Contributor

thanks @wsalembi

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.

2 participants