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

Fix cleanup of securitySchemes #14

Merged
merged 4 commits into from
Dec 11, 2024
Merged

Conversation

dolmen
Copy link
Member

@dolmen dolmen commented Dec 8, 2024

Fixes #10 by implementing the correct algorithm to lookup for references to /components/securitySchemes.

Cc: @pisaychuk, @macr1408

@dolmen dolmen self-assigned this Dec 8, 2024
Add iterObjectPtr, propertyPtr, iterPropertyPtr.
Refactor iterPaths to use iterObjectPtr.
Fix lookup of references to securitySchemes before cleaning it.

/components/securitySchemes are not referenced by $ref like other components.
We now implement the correct algorithm to lookup for references before cleaning:
- look for references in Security Requirements in /security
- look for references in Security Requirements in /paths/<path>/<method>/security
@dolmen dolmen force-pushed the fix-cleanup-of-securitySchemes branch from dc0073b to c16c459 Compare December 8, 2024 23:13
@dolmen dolmen merged commit 1758e7f into master Dec 11, 2024
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.

securitySchemes is being cleared
1 participant