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

Only filter out actuator endpoints with double asterisks. Fixes #2895 #2896

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

mc1arke
Copy link
Contributor

@mc1arke mc1arke commented Feb 8, 2025

All endpoints that contain a double-asterisks in the path are currently being excluded from the output Swagger definition, rather than just excluding match-all actuator endpoints. The endpoint matching is therefore being altered to pre-filter the actuator endpoints to remove any that contain a double-asterisks in the path, and remove the subsequent filtering of double asterisks in the combined endpoints list so that non-actuator endpoints aren't dropped by the actuator filtering.

@mc1arke mc1arke changed the title Only filter out actuator endpoints with double asterisks. Fixes #2817 Only filter out actuator endpoints with double asterisks. Fixes #2895 Feb 8, 2025
…gdoc#2895

All endpoints that contain a double-asterisks in the path are currently
being excluded from the output Swagger definition, rather than just
excluding match-all actuator endpoints. The endpoint matching is
therefore being altered to pre-filter the actuator endpoints to remove
any that contain a double-asterisks in the path, and remove the
subsequent filtering of double asterisks in the combined endpoints list
so that non-actuator endpoints aren't dropped by the actuator filtering.
@bnasslahsen bnasslahsen merged commit 6db4bbc into springdoc:main Feb 13, 2025
1 check passed
@bnasslahsen
Copy link
Collaborator

Thank you @mc1arke for your contributions to the project 👍

@mc1arke mc1arke deleted the issue/2895 branch February 13, 2025 09:16
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