search

lrlna / pino-colada

cute ndjson formatter for pino 🌲🍹
MIT License
245 stars 46 forks source link

Added support for pino-http and hapi-pino #9

Closed mcollina closed 7 years ago

mcollina commented 7 years ago

As titled :).

I've done a bunch of changes, let me know if you want things to be handled differently. In pino-http there is no "content length" concept, but I hope to add it there asap.

cc @davidmarkclements

davidmarkclements commented 7 years ago

extra context - pino-http support is also express-pino-logger and express-koa-logger support

lrlna commented 7 years ago

Ohhhh this is rad @mcollina!!

I am going to look into it a bit more tomorrow cause I am still sick in bed today, but on the first glance ✌️✌️✌️

On Wed, Mar 29, 2017, 15:49 David Mark Clements, notifications@github.com wrote:

extra context - pino-http support is also express-pino-logger and express-koa-logger support

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/lrlna/pino-colada/pull/9#issuecomment-290114404, or mute the thread https://github.com/notifications/unsubscribe-auth/AHu3CIZFzTq-z31NlsgeKyqVgrmn2zVqks5rqm-SgaJpZM4MtEvg .

lrlna commented 7 years ago

okay, had a chance to properly look through! All looks good; just a little comment on one of the examples ^___^

mcollina commented 7 years ago

@lrlna updated :)

lrlna commented 7 years ago

meow-excellent! ✨

lrlna commented 7 years ago

v1.2.3!