search

bithavoc / express-winston

express.js middleware for winstonjs
https://www.npmjs.com/package/express-winston
MIT License
797 stars 187 forks source link

`requestWhitelist.push('body')` does not work #136

Closed oleiba closed 7 years ago

oleiba commented 7 years ago

As per the latest version 2.1.2, requestWhitelist.push('body') does not bring to displaying the request's body. In 2.1.0 it works. I assume the break occurred in the version bump to 2.1.1.

kimmajor commented 7 years ago

Hi, I had the same issue. After the fix in https://github.com/bithavoc/express-winston/commit/dc4891f45a920c6db3c1e5427b04a9ff4a783b13 it seems that you must have either a body whitelist or a body blacklist Adding an arbitrary (non-existent) property to the black list will log all body props. options.bodyBlacklist = ['noop']

rosston commented 7 years ago

Fixed and released in 2.1.3.