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

Adding the ability to blacklist the logging of certain headers in the #22245

Closed
wants to merge 3 commits into from
Closed

Adding the ability to blacklist the logging of certain headers in the #22245

wants to merge 3 commits into from

Conversation

nathan-me-lawrence
Copy link

AbstractRequestLoggingFilter class in spring-web

Issue: #22244 Issue Link

So issue SPR-14245 requested the ability to log all headers coming in with a request be added to AbstractRequestLoggingFilter.

There are sometime headers that it would be beneficial not to log. Such as Authorization headers containing jwt tokens.

It would be useful to be able to provide a blacklist of some headers which are then not appended to the log message builder.

AbstractRequestLoggingFilter class in spring-web

Issue: #22244
@pivotal-issuemaster
Copy link

@OhDearMoshe Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@OhDearMoshe Thank you for signing the Contributor License Agreement!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Jan 24, 2019
@nathan-me-lawrence
Copy link
Author

Closing as the issue has been fixed by someone else

@rstoyanchev rstoyanchev added status: superseded An issue that has been superseded by another and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Mar 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: superseded An issue that has been superseded by another
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants