datalust / winston-seq

A Winston v3 transport for Seq
Apache License 2.0
13 stars 3 forks source link

Basic implementation #3

Closed rj-xy closed 3 years ago

rj-xy commented 3 years ago

This is the implementation without using streams. I've included an integration test, to prove it works

nblumhardt commented 3 years ago

Great! I should be able to take a proper look through this later today 👍

nblumhardt commented 3 years ago

Ah - I see -

https://github.com/datalust/seq-logging/blob/dev/seq_logger.js#L153

I'll PR a change into seq-logging to remove the level name restriction 👍

rj-xy commented 3 years ago

Ah - I see -

https://github.com/datalust/seq-logging/blob/dev/seq_logger.js#L153

I'll PR a change into seq-logging to remove the level name restriction +1

Oh! I understand now too - seq-logging will just do a pass-through for level, then the seq server will do its best to parse/match. Thanks :+1: