Overriding HttpClientHandler.SendAsync() causes StackOverflowException about dd-trace-dotnet

[lucaspimentel]

Definitely an issue we need to fix. Thanks for testing by disabling the HttpMessageHandler integration. Disabling it is a temporary workaround. While that integration is disabled, you won't get traces for any outbound HTTP requests that use HttpMessageHandler class (e.g. HttpClient).

[lucaspimentel]

Thanks for the detailed report! We suspect this may be related to WindowsAzure.Storage using custom implementation of HttpClientHandler. As a quick test to narrow down the issue, can you set the following environment variable in your Dockerfile to disable that integration while leaving everything else enabled?

DD_DISABLED_INTEGRATIONS=HttpClientHandler

[jonhoare]

Thanks for getting back to me.

I have just tried your suggestion, rebuilt and re-run the docker file and still the same issue I'm afraid.

[lucaspimentel]

Sorry, my mistake. The integration name is HttpMessageHandler, not HttpClientHandler. (I should have referenced the docs 🤦‍♂)

DD_DISABLED_INTEGRATIONS=HttpMessageHandler

[jonhoare]

Ahh that did indeed work!

So is that the work around in this scenario or does this highlight an issue for a potential fix in the dd-trace-dotnet client at some point?

[lucaspimentel]

This issue should now be resolved with release 1.3.