datalust / seq-input-gelf

Ingest GELF payloads into Seq
https://datalust.co/seq
Apache License 2.0
15 stars 8 forks source link

include sources in error messages and don't overflow on invalid timestamps #113

Closed KodrAus closed 1 year ago

KodrAus commented 2 years ago

For #111 and #112

This also includes some noise from formatting changes. There are 2 notable changes in here:

  1. Don't panic when timestamps from GELF would overflow when converted into an Instant. This can happen if the number sent isn't at second-precision.
  2. Include the messages from source errors, not just the top-level one, when GELF processing fails.
KodrAus commented 2 years ago

This needs https://github.com/datalust/seqcli/pull/257 released before it'll work.