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

Feat: Option to include headers #27

Open
hnrchrdl opened this issue Dec 8, 2021 · 1 comment
Open

Feat: Option to include headers #27

hnrchrdl opened this issue Dec 8, 2021 · 1 comment

Comments

@hnrchrdl
Copy link

hnrchrdl commented Dec 8, 2021

I would like to have an option for specifically including headers. This would give better control over the headers that are logged.

If you like, i can file a PR for this.

@tellnes
Copy link
Owner

tellnes commented Dec 18, 2021

You could archive this with a custom request serializer. But sure, I'm open to a PR. What about a new config option named includeHeaders?

obscureHeaders and excludeHeaders should probably take presedens over this. So this can easily be implemented by adding some code after the base serialisation code and before the code that applies those.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants