-
Notifications
You must be signed in to change notification settings - Fork 240
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
add feature - track header data for ajax & fetch requests' response and put to part c properties bag #946
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also include in your PR a screenshot of the result in the Portal. as well as a description of what it does? off-by-default, etc
vNext/extensions/applicationinsights-dependencies-js/src/ajax.ts
Outdated
Show resolved
Hide resolved
vNext/extensions/applicationinsights-dependencies-js/src/ajax.ts
Outdated
Show resolved
Hide resolved
vNext/extensions/applicationinsights-dependencies-js/src/ajax.ts
Outdated
Show resolved
Hide resolved
vNext/shared/AppInsightsCore/src/JavaScriptSDK.Enums/LoggingEnums.ts
Outdated
Show resolved
Hide resolved
vNext/extensions/applicationinsights-dependencies-js/src/ajax.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will this also track correlation headers that this SDK sets on http requests?
Adding feature of tracking response headers data of ajax and fetch requests in dependency plugin. This feature is off by default. It can be turned on by setting the config flag "enableHeaderTracking: true" when initializing the ApplicationInsights instances. All data are stored in customer properties bag.
Example of Fetch request result in Azure Portal:
Example of Ajax request result in Azure Portal:
Related issue: #736