Skip to content

allOperationIds does not consider webhooks #392

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
Closed
mattpolzin opened this issue Feb 12, 2025 · 1 comment · Fixed by #393
Closed

allOperationIds does not consider webhooks #392

mattpolzin opened this issue Feb 12, 2025 · 1 comment · Fixed by #393
Labels
bug Something isn't working v3.x

Comments

@mattpolzin
Copy link
Owner

mattpolzin commented Feb 12, 2025 •
edited
Loading

This was by design originally but it was also a decision made when webhooks weren't a document property rather than a decision made because webhooks shouldn't be considered.

Semantically, I'm considering this omission a bug because "all operation ids" should mean all of them.
@mattpolzin mattpolzin mentioned this issue Feb 12, 2025
Draft
@mattpolzin mattpolzin added bug Something isn't working v3.x labels Feb 12, 2025
@mattpolzin
Copy link
Owner Author

The same bug applies to allServers and allTags.

@mattpolzin mattpolzin mentioned this issue Feb 12, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working v3.x
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bugfix/392/capture all operation ids mattpolzin/OpenAPIKit
1 participant
@mattpolzin