search

datalust / pino-seq

A stream to send Pino events to Seq
Apache License 2.0
11 stars 10 forks source link

Correctly implemented NodeJS stream #2

Closed stefan-jonasson closed 4 years ago

stefan-jonasson commented 4 years ago

The seq stream wrapper is now implemented according to https://nodejs.org/api/stream.html#stream_implementing_a_writable_stream

Also added support for capturing messages not correctly formatted. This is useful when doing out-of-process logging to capture a nodejs crash (SEGFAULT, which cannot be handled with a in-process handler).

Fixed the logging of error objects. It now sends the trace in exception property.

tsimbalar commented 4 years ago

@stefan-jonasson thanks a lot, that seems to fix a few of the issue I found while playing around with this a few weeks ago ❤️

tsimbalar commented 4 years ago

I think that in addition, it fixes #1 ? :)

larenelg commented 4 years ago

Indeed it does @tsimbalar thank you!