Stijn Moreels

Results 140 comments of Stijn Moreels

Some info related to this and migration from App Insights client SDK to OpenTelemetry using service-to-service correlation/distributed tracing. Minimalistic setup: ```csharp var builder = new HostBuilder(); builder.ConfigureServices(services => { services.ConfigureOpenTelemetryTracerProvider((provider,...

> I can pick this one up That would be great, thx a lot!

> @stijnmoreels > > > We should use `ArgumentException` when checking for blank strings, not `ArgumentNullException`. > > Only when checking for blank **strings**? We can fallback on `ArgumentException` for...

Hi @HLobik63 , great for your use of Arcus. The problem you're refering too, is expected behavior. The `.UseHttpCorrelation()` is only meant to be used in HTTP-scenarios, hence the exception....

Backwards compatibility reasons is the main one. In Arcus.Testing, this already removed, but other Arcus repo's there might indeed some chore work to be done.

> So it's okay to remove but hasn't been removed yet? It needs to be discussed first. Arcus.Testing had the luxuary that it didn't build on top of .NET Standard...

It is not using the .NET 10 version throughout all Arcus packages, though, maybe first these need to be streamlined so they are all using .NET 10 before making conclusions....

@fgheysels , best to check this PR before other ones, as it will greatly improve the stability of parallel integration test runs.

There are multiple providers defined, so we don't force people to use key vault. I even believe that by the name, we kinda suggest this to be an environment variable....

> If it's retrieved from KeyVault, this won't work as KeyVault doesn't allow underscores in its secret-name. Since a KeyVault secretprovider is added via the template , I suggest that...