Hello,

It would be beneficial to extend the AbstractRequestLoggingFilter to have a queryPredicate, similar to Predicate<String> headerPredicate to allow for the redaction of any sensitive/restricted data.

What do you think? Would this be an acceptable addition?

I am happy to take this on, provided the idea is accepted.

Comment From: sbrannen

Are you proposing that such a predicate would be used to filter individual query parameters by name?

If so, then queryParamPredicate would be a better name for the predicate.

Comment From: sic2

Are you proposing that such a predicate would be used to filter individual query parameters by name?

If so, then queryParamPredicate would be a better name for the predicate.

Yes. Happy with that name too.

Comment From: sbrannen

I am happy to take this on, provided the idea is accepted.

Feel free to submit a PR! 👍

Note, however, that we plan to release 7.0 RC1 next week. So if you can get the work done before mid next week, that would be great.

Otherwise, we might take it over.

Please let us know your plans.

Comment From: sic2

@sbrannen Thanks! I am planning to work on this during the weekend. Hope to get a PR out soon.