Skip to content

Commit

Permalink
fix(opentelemetry-core): confusing log extract of composite propagator (
Browse files Browse the repository at this point in the history
  • Loading branch information
rv2673 authored Sep 26, 2024
1 parent 9de3151 commit a5dade5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ For semantic convention package changes, see the [semconv CHANGELOG](packages/se

* fix(sdk-trace-base): avoid keeping non-string `status.message` on `Span#setStatus()` [#4999](https://github.com/open-telemetry/opentelemetry-js/pull/4999) @pichlermarc
* fix(sdk-metrics): Add missing catch and handle error in promise of `PeriodicExportingMetricReader` [#5006](https://github.com/open-telemetry/opentelemetry-js/pull/5006) @jj22ee
* fix(opentelemetry-core): confusing log extract of composite propagator [#5017](https://github.com/open-telemetry/opentelemetry-js/pull/5017) @rv2673

### :books: (Refine Doc)

Expand Down
2 changes: 1 addition & 1 deletion packages/opentelemetry-core/src/propagation/composite.ts
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ export class CompositePropagator implements TextMapPropagator {
return propagator.extract(ctx, carrier, getter);
} catch (err) {
diag.warn(
`Failed to inject with ${propagator.constructor.name}. Err: ${err.message}`
`Failed to extract with ${propagator.constructor.name}. Err: ${err.message}`
);
}
return ctx;
Expand Down

0 comments on commit a5dade5

Please sign in to comment.