Vishwesh Bankwar
Vishwesh Bankwar
Adding initial analysis (needs to be validated). Two possible approaches: Both the approaches need accessing TelemetryConfiguration object to get TelemetryProcessorChain 1. ASP.NET Core (HttpClient): configuration field inside https://github.com/microsoft/ApplicationInsights-dotnet/blob/main/WEB/Src/DependencyCollector/DependencyCollector/HttpCoreDiagnosticSourceListener.cs 2. ASP.NET...
> @alanwest I could live with "Shim" but I think "Adapter" could also work. In the case of Serilog & EventSource, maybe even "Appender" since the spec has run with...
closing this as the changes are tracked with https://github.com/open-telemetry/opentelemetry-dotnet/issues/4484
This issue is now fixed in 1.4.x. Note that `IsRemote=true` for this to work. https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry/CHANGELOG.md#:~:text=TracerProviderSDK%20modified%20for%20spans%20with%20remote%20parent.%20For%20such%20spans%20activity%20will%20be%20created%20irrespective%20of%20SamplingResult%2C%20to%20maintain%20context%20propagation.%20(%233329
fixed in https://github.com/open-telemetry/opentelemetry-dotnet/pull/5026
https://github.com/open-telemetry/semantic-conventions/blob/main/docs/http/http-metrics.md#metric-httpclientrequestduration
https://www.nuget.org/packages/Microsoft.ApplicationInsights/2.22.0-beta3 has a fix for #2742. @dracos1993 Could you try this out to see if it resolves your issue as well?
Linking a similar issue from OTel side: https://github.com/open-telemetry/opentelemetry-dotnet-contrib/issues/1765 cc: @noahfalk @samsp-msft
@specialforest This feature is currently in the pipeline; however, there is no ETA at the moment. Will share the updates once there is progress.
FYI - this feature depends on https://github.com/open-telemetry/opentelemetry-dotnet-contrib/issues/1585 and backend readiness. @specialforest Please follow https://github.com/open-telemetry/opentelemetry-dotnet-contrib/issues/1585 for further updates. I am closing this one.