datalust / seq-forwarder

Local collection and reliable forwarding of log data to Seq
Apache License 2.0
57 stars 15 forks source link

A major refresh #51

Closed nblumhardt closed 3 years ago

nblumhardt commented 3 years ago

While Seq Forwarder is a critical piece of the Seq ecosystem, we've had difficulty maintaining forward momentum on this project:

This PR is a line in the sand - a bit of a fresh start - that modernizes the codebase and trims it back to the bare essentials necessary to meet Seq Forwarder's goal, which is:

Local collection and reliable forwarding of log data to Seq

To get there I've:

Some of these items will be a step backwards, but, by simplifying the project and improving the development experience we can put our effort into improving the features, efficiency, reliability, and manageability of the core log forwarder.

Fixes #16, fixes #21, fixes #49, fixes #50

darrynparker commented 3 years ago

Yep - this looks great and I love the .Net Core implementation. I had a look over the weekend and so far it fills the gap......but the only other feature gap is the capability of the original consuming application to have it's own Seq API key also forwarded to the Seq server.

Apart from that - well done.

nblumhardt commented 3 years ago

Hi @darrynparker - thanks for taking a look!

I just tried out API key forwarding and it still works for me using the 2.0 preview release - if you're finding otherwise, it would be great to have a bug report (I could be missing something :-) )

Thanks!

darrynparker commented 3 years ago

OK - thanks, I'll check it out.