-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Is there any way to replace the feign header 's value as "secret"? #1561
Comments
if you want to exclude secret values from the logs, why not configure your logger to mask those. We use |
I use logbook for project but logbook. removeheader does not affect feign log ,because feign log use slf4J by default when it is logging.And So I can see sensitive data in log when request sent. |
I found (https://github.com/OpenFeign/feign/pull/1530)this .It is exculude but I want to replace with"secret" |
@MGunel Logback can be used as implementation of slf4j: you can configure excluding as @sakibulalam said. For me it is better way than to add masking logic to Feign's Logger. |
Echoing @radio-rogal, You will need to provide your own |
Given there's been no update or proposal, I'm assuming you are good. |
Is there any way to replace the feign header's value in log ? I found the way to exclude headers from logs but I want to replace feign header's value as "secret" .I did this by overriding shouldLogRequestHeader method
The text was updated successfully, but these errors were encountered: