Skip to content
This repository has been archived by the owner on Jun 10, 2020. It is now read-only.

Make W3C Correlation default and leverage native W3C support from new System.Diagnostics.DiagnosticSource Activity #964

Merged
merged 35 commits into from
Aug 27, 2019

Conversation

cijothomas
Copy link
Contributor

AspNetCore RequestCollection specific changes, followup for the base SDK changes introduced in the PR:
microsoft/ApplicationInsights-dotnet#1193

Also Fixes:
#956
#900

To do:
3.XX support (separate PR)

(replaying #958 as it had to be reverted temporarily)

@cijothomas cijothomas merged commit 73998d3 into develop Aug 27, 2019
@cijothomas cijothomas mentioned this pull request Aug 29, 2019
@cijothomas
Copy link
Contributor Author

@reyang Please have a look.

@@ -63,4 +53,4 @@ steps:
inputs:
PathtoPublish: "$(build.artifactstagingdirectory)"
ArtifactName: "drop"
ArtifactType: "Container"
ArtifactType: "Container"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit, empty line before EOF

@@ -1,5 +1,10 @@
# Changelog

## Version 2.8.0-beta3
- [Make W3C Correlation default and leverage native W3C support from Activity.](https://github.com/microsoft/ApplicationInsights-aspnetcore/pull/958)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To be precise, this is "W3C TraceContext" instead of "W3C Correlation".

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants