This repository has been archived by the owner on Jul 5, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 67
Support W3C distributed tracing standard #945
Merged
Merged
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
ffcff4c
Support W3C propagation in the process and http injection
c828db3
Set context by default
cfdc21f
fixes
bf8a8c4
Enable W3C on desktop: requests and dependencies
070fe7d
fix broken tests
57ef672
helpers and fixes
10dc70a
fix no w3c context in parent
f330c04
changelog and more tests
603e925
review comments, tracestate proper ordering and tracestate in the pro…
43a344a
0 default sampling
0cb176a
correlation backward compatibility via legacy ids in custom dimensions
fa60df5
Enable W3C via setting, not env var
a91c71a
minor fixes
2ebed64
fix stylecop
aec4fc4
limit visibility in common items
5c6b998
public w3c stuff
8d4f703
minor fixes
61aac47
minor fixes
f627813
some validation
34cdab0
more validations and code review
4f229b1
Tracestate and Traceparent
6bcad79
@ before msappid
78b4573
Merge branch 'develop' into lmolkova/W3CSupportOnCore
08ea8c5
rename GetTraceState to GetTracestate
7d67622
Merge branch 'lmolkova/W3CSupportOnCore' of https://github.com/Micros…
5735f66
traceid and spanid validation, az alias for all azure trace states
84a561c
fix separators
82faad3
Merge branch 'develop' into lmolkova/W3CSupportOnCore
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I like hiding this class from public surface. What's the reason to keep it public?
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.
this class is used by AspNetCore SDK and should be visible to it. There are two alternatives: