Nikita Balabaev

Results 34 comments of Nikita Balabaev

Okay, since it's by design to reduce risk of a privacy incidents, I would go with closing this issue. @sjwaight feel free to reopen it if you have other concerns...

Regarding redaction, my suggestion would be to register `NullRedactorProvider` if `IRedactorProvider` isn't registered in the DI yet. But we need to ensure that subsequent call to `AddRedaction()` replaces previous registration.

Hello @ZzZombo, I was just trying to reproduce the issue but had no luck: ![image](https://github.com/dotnet/extensions/assets/6381023/322a7d60-d7ae-4b5e-9ad4-be23b839d8cd) Are there any specific steps I also need to take? I'm using .NET SDK `8.0.200`...

Okay, you missed the usage of `[LoggerMessage]` in your reproduction steps. Here's the minimal repro: ```xml net8.0-windows WinExe true ``` And `Log.cs`: ```cs using Microsoft.Extensions.Logging; internal static partial class Log...

Even if I replace `Microsoft.Extensions.Resilience` reference with `Microsoft.Extensions.Telemetry.Abstractions` package, it still emits the same error

> It should work universally. @RussKie my concern was that `RemoveDuplicateAnalyzers` target might exist for WPF apps only

> Reading the comment [#4969 (comment)](https://github.com/dotnet/extensions/issues/4969#issuecomment-1991870300) it suggests the issue is not specific to extension telemetry library and it suggest the fix should be somewhere else maybe in SDK targets?...