Skip to content
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

feat: Add B3format propagator #208

Merged
merged 5 commits into from
Aug 30, 2019

Conversation

mayurkale22
Copy link
Member

Which problem is this PR solving?

Short description of the changes

@codecov-io
Copy link

codecov-io commented Aug 19, 2019

Codecov Report

Merging #208 into master will increase coverage by 1.39%.
The diff coverage is 99.38%.

@@            Coverage Diff             @@
##           master     #208      +/-   ##
==========================================
+ Coverage   82.65%   84.05%   +1.39%     
==========================================
  Files          52       54       +2     
  Lines        1776     1938     +162     
  Branches      125      151      +26     
==========================================
+ Hits         1468     1629     +161     
- Misses        308      309       +1
Impacted Files Coverage Δ
...s/opentelemetry-core/test/context/B3Format.test.ts 100% <100%> (ø)
...telemetry-core/src/context/propagation/B3Format.ts 97.82% <97.82%> (ø)

Copy link
Member

@bg451 bg451 left a comment

Choose a reason for hiding this comment

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

sweet looks good to me

}

/**
* Propagator for the B3 HTTP header format.
Copy link
Member

Choose a reason for hiding this comment

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

Maybe note that this is for multiple headers, not single headers (since the b3 spec mentions both ways of propagating)

Copy link
Contributor

@danielkhan danielkhan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@OlivierAlbertini OlivierAlbertini left a comment

Choose a reason for hiding this comment

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

LGTM

@mayurkale22 mayurkale22 merged commit b9c9212 into open-telemetry:master Aug 30, 2019
@mayurkale22 mayurkale22 deleted the b3format branch August 30, 2019 17:18
pichlermarc pushed a commit to dynatrace-oss-contrib/opentelemetry-js that referenced this pull request Dec 15, 2023
… span name (open-telemetry#208)

* fix: ensure span name is a string to avoid [object Object] as span name

Signed-off-by: Naseem <[email protected]>

* fix: make test read better

Signed-off-by: Naseem <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add b3 propagator
5 participants