search

elastic / ecs-dotnet

https://www.elastic.co/guide/en/ecs-logging/dotnet/current/setup.html
Apache License 2.0
117 stars 59 forks source link

Making logger provider the sole manager of channel lifecycle. #394

Closed moikmellah closed 4 months ago

moikmellah commented 4 months ago

Fixes a bug where ElasticsearchLoggerProvider passes a reference to EcsDataStreamChannel when instantiating ElasticsearchLogger and disposes of the channel on configuration reload events, leading to ElasticsearchLogger instances hanging on to a disposed channel and silently dropping log events.

cla-checker-service[bot] commented 4 months ago

❌ Author of the following commits did not sign a Contributor Agreement: 7f31e956ba815123afbbda7029fe12bfba943b85

Please, read and sign the above mentioned agreement if you want to contribute to this project

moikmellah commented 4 months ago

CLA submitted, please let me know if anything else is needed. Thanks!