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

securityScheme not included in reduced API definition #860

Closed
devishke-orange opened this issue Aug 16, 2023 · 3 comments · Fixed by readmeio/oas#791 or #867
Closed

securityScheme not included in reduced API definition #860

devishke-orange opened this issue Aug 16, 2023 · 3 comments · Fixed by readmeio/oas#791 or #867
Assignees
@erunion
Labels
command:openapi Issues pertaining to the `openapi`, `validate`, `reduce`, or `swagger` commands question Further information is requested

Comments

@devishke-orange
Copy link

Hi. I use the reduce functionality you guys provide to break up our API spec into smaller files and push them to ReadMe.

I have a securitySchemes defined in my components and also put the security array in the root level. After the reduction process rdme openapi:reduce (reducing by tags) it includes the security array but not the securitySchemes object.

Is there something I'm missing?
@kanadgupta kanadgupta added question Further information is requested command:openapi Issues pertaining to the `openapi`, `validate`, `reduce`, or `swagger` commands and removed question Further information is requested labels Aug 16, 2023
@kanadgupta

This comment was marked as resolved.

Sign in to view
@erunion
Copy link
Member

erunion commented Aug 18, 2023

I'm able to repro this in our oas library. Working on a fix.

@erunion erunion self-assigned this Aug 18, 2023
@erunion erunion mentioned this issue Aug 18, 2023
Merged
@erunion erunion reopened this Aug 18, 2023
@erunion erunion mentioned this issue Aug 18, 2023
Merged
1 task
@kanadgupta
Copy link
Member

Hi @devishke-orange — I just tagged v8.6.5 which should address this:

https://github.com/readmeio/rdme/releases/tag/8.6.5
https://www.npmjs.com/package/rdme/v/8.6.5

Thanks for reporting!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erunion erunion

Labels
command:openapi Issues pertaining to the `openapi`, `validate`, `reduce`, or `swagger` commands question Further information is requested
Projects
None yet
Milestone
No milestone
3 participants
@erunion @kanadgupta @devishke-orange