fix: msg property not logged if log level isn't a number

lrlna / pino-colada

cute ndjson formatter for pino 🌲🍹

MIT License

245 stars 46 forks source link

fix: msg property not logged if log level isn't a number #28

Closed gunjam closed 5 years ago

gunjam commented 5 years ago

Hi!

Using pino-colada all my log messages are printed as undefined because I've got pino set with useLevelLabels to true and pino-colada only prints the msg property if the log level is a number.

I've moved the line which copies msg to message out of the number conversion function to solve this.

Thanks!

lrlna commented 5 years ago

hey @gunjam! This seems reasonable. Thanks for submitting the PR!

lrlna commented 5 years ago

📦 1.4.5