datalust / seq-tickets

Issues, design discussions and feature roadmap for the Seq log server
https://datalust.co/seq
98 stars 5 forks source link

`NullReferenceException` prevents ingestion of OTLP payloads from `Serilog.Sinks.OpenTelemetry` with missing `IncludedData` options #2314

Open nblumhardt opened 2 weeks ago

nblumhardt commented 2 weeks ago

When Serilog.Sinks.OpenTelemetry is used for ingestion, in its default configuration, IncludedData.MessageTemplateText is selected. This leads to Seq accepting the message template value, but if it contains format strings (f in {X:f}), ingestion fails with a NullReferenceException.

Removing IncludedData.MessageTemplateText from the default options is one possible workaround:

options.IncludedData ^= IncludedData.MessageTemplateText;

This is a candidate for a 2024.3 patch, but I'll raise it under 2025.1 until we have a feel for when the next patch might be assembled. In the meantime, please let us know if this affects you.