Nicholas Blumhardt

Results 929 comments of Nicholas Blumhardt

Hi @DevProJSSV; it doesn't look like this will make it into 4.1.0 (just about to publish) but a minimal, tested PR should be fairly quick to get through the works...

Typo in there, mentions Seq while this repo is OTLP. I think this could be confusing right now, because Serilog.Sinks.OpenTelemetry doesn't support SerilogTracing. I'd like to open a discussion around...

I still think this would be way more compelling (and a bit less confusing) if the message was _consider using SerilogTracing with this sink_. I'll scope out and raise that...

Thanks for all the notes. This seems like something to consider from the `Serilog.Settings.Configuration` angle; the configuration system is responsible for resolving parameters for Serilog sinks, so plugging in an...

Hi! I think the intended path for this is https://github.com/serilog/serilog-settings-configuration/issues/398 - and some help over there would be most welcome if you have the opportunity. I can see the place...

See also #123, just merged, which will support the changes in _Serilog.Settings.Configuration_.

Splitting `WriteTo.OpenTelemetry(..)` might need to be split into the one we have today, and one that accepts an `IConfiguration` or similar and no additional args. Trying to meld the two...

Just adding another note for when we're back around to this; the typical way for Serilog to interact with external configuration is via `ReadFrom.X()` where `X()` today is one of...

I think this is ultimately where we're heading, but I'm not sure what the mechanics should be. Since resource attributes are expected to be static for the life of the...