Add an headerBlacklist option

bithavoc / express-winston

express.js middleware for winstonjs

https://www.npmjs.com/package/express-winston

MIT License

796 stars 187 forks source link

Closed maxday closed 4 years ago

maxday commented 5 years ago

Hello,

It could be great to have an headerBlacklist option. This could be an array of headers name to omit from logs.

I know that this can be achievable using the requestFilter but I think that having a specific option will help.

Use case : don't log cookies, authorization bearer tokens etc.

I will prepare a PR for this change

What do you think ?

alokrajiv commented 4 years ago

hi @maxday @bithavoc , think this can be closed now. Since the PR #217 was merged?

bithavoc commented 4 years ago

Released as express-winston@3.3.0. Please notice, there are breaking changes migrating from 3.x to 4.x