Skip to content

Introduce queryParamPredicate in AbstractRequestLoggingFilter #35603

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
wants to merge 1 commit into from
Closed

Introduce queryParamPredicate in AbstractRequestLoggingFilter #35603

wants to merge 1 commit into from
+51 −3

Conversation

sic2
Copy link
Contributor

@sic2 sic2 commented Oct 11, 2025 •
edited by sbrannen
Loading

Addresses:

These changes add a queryParamPredicate to the AbstractRequestLoggingFilter, similar to the headerPredicate.

This can be used to mask given query params.

@sic2 sic2 force-pushed the feat/web/query_param_predicate branch from 770ef61 to d2472b4 Compare October 11, 2025 08:58
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Oct 11, 2025
@sic2 sic2 force-pushed the feat/web/query_param_predicate branch from d2472b4 to 89471e6 Compare October 11, 2025 08:59
@sbrannen sbrannen changed the title Introduce queryParamPredicate in AbstractRequestLoggingFilter Introduce queryParamPredicate in AbstractRequestLoggingFilter Oct 11, 2025
@sbrannen sbrannen self-assigned this Oct 11, 2025
@sbrannen sbrannen added in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Oct 11, 2025
@sbrannen sbrannen added this to the 7.0.0-RC1 milestone Oct 11, 2025
@sbrannen sbrannen mentioned this pull request Oct 11, 2025
Closed
@sbrannen sbrannen closed this in 87a7507 Oct 11, 2025
sbrannen added a commit that referenced this pull request Oct 11, 2025
@sbrannen
Polish contribution
3eef39e 
See gh-35603
@sbrannen
Copy link
Member

This has been merged into main in 87a7507 and slightly revised in 3eef39e.

Thanks for the contribution! 👍

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

Reviewers

No reviews

Assignees

@sbrannen sbrannen

Labels

in: web Issues in web modules (web, webmvc, webflux, websocket) type: enhancement A general enhancement

Projects

None yet

Milestone

7.0.0-RC1

Development

Successfully merging this pull request may close these issues.

3 participants

@sic2 @sbrannen @spring-projects-issues