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

Provide a way to exclude headers from logs #1492

Open
clemstoquart opened this issue Aug 23, 2021 · 3 comments
Open

Provide a way to exclude headers from logs #1492

clemstoquart opened this issue Aug 23, 2021 · 3 comments
Labels
enhancement For recommending new capabilities feedback provided Feedback has been provided to the author

Comments

@clemstoquart
Copy link

Hi feign team,

It would be very useful to be able to exclude some specific headers from the feign http logs. For example we don't want to log authorisation header for security reasons.

Have a nice day.

@ankisinha3005
Copy link

you can create a custom http client and use an interceptors to modify your logs according to how you want.

@clemstoquart
Copy link
Author

Yes you're right. But IMO this is required so often that it would be worth it to add the feature in feign.

@kdavisk6
Copy link
Member

Another option is your own Logger implementation, but I do agree, this would be a good enhancement. We are open to proposals on how to do this.

@kdavisk6 kdavisk6 added enhancement For recommending new capabilities feedback provided Feedback has been provided to the author labels Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement For recommending new capabilities feedback provided Feedback has been provided to the author
Projects
None yet
Development

No branches or pull requests

3 participants