Mark Wolff
Mark Wolff
Which version of the SDK are you using? I'm unable to repro this locally with this sample app. ```ts const Promise = require('bluebird'); global.Promise = Promise; Promise.longStackTraces(); const appInsights =...
Thanks for investigating this, seems good to me. Would you like to submit a PR with this change?
This SDK does not automatically support gRPC, but [OpenTelemetry](http://github.com/open-telemetry/opentelemetry-js) + [Monitor Exporter](https://github.com/Azure/azure-sdk-for-js/tree/master/sdk/monitor/monitor-opentelemetry-exporter) does.
For node.js 8.2+, we use [async_hooks](https://nodejs.org/api/async_hooks.html) which was very new from your node.js version's perspective and so it might be buggy/unstable at that point. You might also have some dangling...
It could be if you are referring to the AI context in your own code at all. I'm unable to repro a leak with a simple express demo on node...
Setting forceCls should have no effect on incoming request telemetry correlation. It reads the http header the same way, regardless of this configuration. What do you see on `req.header["request-id"]` in...
You should import and start the SDK before you import in any HTTP modules. It seems the patching required for CLS-mode requires us to not only be imported before express/requests,...
A manual halt is fine, but maybe an upper limit on the amount of retries would be better so that others in a similar situation don't need to make code...
As Osvaldo mentioned, the best way seems to bake your own class which can extend the `Sender` class. You can implement custom behavior based on payload age (on any retriable...