Nicholas Blumhardt

Results 455 comments of Nicholas Blumhardt

I'm not keen on dropping the dependencies here, not least because of the downstream breakage, but also because this creates one more barrier to adopting Serilog: the default .NET web...

Thanks for the report! Someone will need to set up "break on all exceptions", or step through the test in a debugger to narrow it down. I assembled a test...

Does the issue persist if you switch from `CreateBootstrapLogger()` to `CreateLogger()`?

@pikoscielniak thanks for helping to track it down. `CreateBootstrapLogger()` creates a `ReloadableLogger`, which the `UseSerilog()` callback reconfigures and then freezes. Freezing is a stateful operation: https://github.com/serilog/serilog-extensions-hosting/blob/dev/src/Serilog.Extensions.Hosting/Extensions/Hosting/ReloadableLogger.cs#L80 Not sure how best...

https://nblumhardt.com/2020/10/bootstrap-logger/#why-cant-we-just-do-both has some info on the problems `CreateBootstrapLogger()` resolves; if those issues don't apply to you then it's fine to just use `CreateLogger()` instead.

@span @MichaelDeutschCoding sounds like you're hitting something else; if you're sure it's Serilog-related, could you please post a new issue and include as much detail as possible? Thanks!

Thanks for the heads-up! This looks like it's due to `EventSourceLoggerProvider` holding on (permanently?) to all of the `EventSourceLogger`s that it creates: https://github.com/dotnet/runtime/blob/main/src/libraries/Microsoft.Extensions.Logging.EventSource/src/EventSourceLoggerProvider.cs#L36 I'm not sure why this design is...

Thanks @spolaore and @raphaabreu 👍 I think we'll want to take a similar approach to MEL, here. Yes, Serilog calls the other providers directly (it completely replaces `LoggerFactory` when enabled)....

@ivanmonteiro not sure what's going on in your case, but if you're load testing with a lot of concurrency, it could just be that `System.Console` can't keep up. First step...