-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added the ability to track dependency calls #46
Conversation
@kentcb any chance to get an approval for this PR or do you need anything? |
@kentcb any chance, you could take a look at this PR? Should be a non breaking extension. |
@johnnyggalt awesome! It could easily be that this would not be necessary. The v2 JS SDK sent the header by default, I just saw that in the v3 SDK it is no longer included by default. |
@m-gug Got it, thanks. I've removed it for now, since as best I can tell it is an internal thing, and worst case scenario is it might cause issues for folks with strict middleware. Anyway, if it turns out to be necessary, it can always be addressed separately to this PR. |
@all-contributors Please add @m-gug for code |
I've put up a pull request to add @m-gug! 🎉 |
Thank you, and no problem - i will check if the telemetry correlation is still working as intended on my side. |
@kentcb do you already have a plan when you will release the new version to pub.dev? |
@m-gug I intend following up on the remaining open issue before releasing. Can't promise any dates though. |
Remote dependency calls can now be tracked with trackDependency().
Background:
With trackRequest I had problems with distributed tracing across multiple application insights instances. AI could not connect the individual entries across the instances.
According to the Application Insights documentation, dependencies should be used to track calls to an HTTP endpoint. Requests are more intended to track incoming requests in the backend.
(https://learn.microsoft.com/en-us/azure/azure-monitor/app/data-model-complete#dependency)
With this change, distributed tracing also worked without any problems.