-
Notifications
You must be signed in to change notification settings - Fork 821
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
Set correct default propagator #869
Labels
good first issue
Good for newcomers
Comments
dyladan
added
bug
Something isn't working
nonconforming
and removed
bug
Something isn't working
labels
Mar 17, 2020
AFAIK current default propagator is a HttpTraceContext propagator, right? |
As I understand it this means the API default not the SDK default. You linked the SDK default. |
mayurkale22
added
up-for-grabs
Good for taking. Extra help will be provided by maintainers
good first issue
Good for newcomers
labels
Mar 17, 2020
It also implies the "noop" implementation needs to propagate context without creating new spans... |
I would like to claim this issue. |
mayurkale22
removed
the
up-for-grabs
Good for taking. Extra help will be provided by maintainers
label
Jun 12, 2020
Closed via #1218 |
pichlermarc
pushed a commit
to dynatrace-oss-contrib/opentelemetry-js
that referenced
this issue
Dec 15, 2023
* feat(mongodb4): added mongodb4 instrumentation * feat(mongodb4): updated mongo hostname * feat(mongodb4): fixed Amir's CR * feat(mongodb4): fixed Amir's CR
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently the default propagator is a noop propagator. According to the spec:
The text was updated successfully, but these errors were encountered: