From a0f984164abc44caa5a6a2156df5ae44cf9b7b49 Mon Sep 17 00:00:00 2001 From: Marc Pichler Date: Tue, 21 May 2024 15:12:44 +0200 Subject: [PATCH] Revert "feat(sdk-trace-node): support xray propagator (#4602)" This reverts commit 75d88f71269b430afbed85999881785ee565ad35. --- CHANGELOG.md | 1 - package-lock.json | 3 --- packages/opentelemetry-sdk-trace-node/package.json | 1 - .../opentelemetry-sdk-trace-node/src/NodeTracerProvider.ts | 2 -- .../test/NodeTracerProvider.test.ts | 3 +-- packages/opentelemetry-sdk-trace-node/tsconfig.json | 3 --- 6 files changed, 1 insertion(+), 12 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 67c447ab70c..ffe241e8f36 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -12,7 +12,6 @@ For experimental package changes, see the [experimental CHANGELOG](experimental/ ### :rocket: (Enhancement) * feat: support node 22 [#4666](https://github.com/open-telemetry/opentelemetry-js/pull/4666) @dyladan -* feat(sdk-trace-node): support `xray` Propagator via `OTEL_PROPAGATORS` environment variable [#4602](https://github.com/open-telemetry/opentelemetry-js/pull/4602) @anuraags ### :bug: (Bug Fix) diff --git a/package-lock.json b/package-lock.json index 5f78c96d3c8..853795ee488 100644 --- a/package-lock.json +++ b/package-lock.json @@ -4772,7 +4772,6 @@ } }, "experimental/packages/propagator-aws-xray-lambda": { - "name": "@opentelemetry/propagator-aws-xray-lambda", "version": "0.51.0", "license": "Apache-2.0", "dependencies": { @@ -35255,7 +35254,6 @@ "dependencies": { "@opentelemetry/context-async-hooks": "1.24.0", "@opentelemetry/core": "1.24.0", - "@opentelemetry/propagator-aws-xray": "1.24.0", "@opentelemetry/propagator-b3": "1.24.0", "@opentelemetry/propagator-jaeger": "1.24.0", "@opentelemetry/sdk-trace-base": "1.24.0", @@ -45171,7 +45169,6 @@ "@opentelemetry/api": ">=1.0.0 <1.9.0", "@opentelemetry/context-async-hooks": "1.24.0", "@opentelemetry/core": "1.24.0", - "@opentelemetry/propagator-aws-xray": "1.24.0", "@opentelemetry/propagator-b3": "1.24.0", "@opentelemetry/propagator-jaeger": "1.24.0", "@opentelemetry/resources": "1.24.0", diff --git a/packages/opentelemetry-sdk-trace-node/package.json b/packages/opentelemetry-sdk-trace-node/package.json index f2b59159c79..508eaa8bf9d 100644 --- a/packages/opentelemetry-sdk-trace-node/package.json +++ b/packages/opentelemetry-sdk-trace-node/package.json @@ -68,7 +68,6 @@ "@opentelemetry/context-async-hooks": "1.24.0", "@opentelemetry/core": "1.24.0", "@opentelemetry/propagator-b3": "1.24.0", - "@opentelemetry/propagator-aws-xray": "1.24.0", "@opentelemetry/propagator-jaeger": "1.24.0", "@opentelemetry/sdk-trace-base": "1.24.0", "semver": "^7.5.2" diff --git a/packages/opentelemetry-sdk-trace-node/src/NodeTracerProvider.ts b/packages/opentelemetry-sdk-trace-node/src/NodeTracerProvider.ts index c82b600d93e..9d552162f64 100644 --- a/packages/opentelemetry-sdk-trace-node/src/NodeTracerProvider.ts +++ b/packages/opentelemetry-sdk-trace-node/src/NodeTracerProvider.ts @@ -26,7 +26,6 @@ import { import * as semver from 'semver'; import { NodeTracerConfig } from './config'; import { JaegerPropagator } from '@opentelemetry/propagator-jaeger'; -import { AWSXRayPropagator } from '@opentelemetry/propagator-aws-xray'; /** * Register this TracerProvider for use with the OpenTelemetry API. @@ -51,7 +50,6 @@ export class NodeTracerProvider extends BasicTracerProvider { () => new B3Propagator({ injectEncoding: B3InjectEncoding.MULTI_HEADER }), ], ['jaeger', () => new JaegerPropagator()], - ['xray', () => new AWSXRayPropagator()], ]); constructor(config: NodeTracerConfig = {}) { diff --git a/packages/opentelemetry-sdk-trace-node/test/NodeTracerProvider.test.ts b/packages/opentelemetry-sdk-trace-node/test/NodeTracerProvider.test.ts index a0acf0d0d02..1b1dbbd6d8f 100644 --- a/packages/opentelemetry-sdk-trace-node/test/NodeTracerProvider.test.ts +++ b/packages/opentelemetry-sdk-trace-node/test/NodeTracerProvider.test.ts @@ -236,7 +236,7 @@ describe('NodeTracerProvider', () => { }); it('should allow propagators as per the specification', () => { - (process.env as any).OTEL_PROPAGATORS = 'b3,b3multi,jaeger,xray'; + (process.env as any).OTEL_PROPAGATORS = 'b3,b3multi,jaeger'; const provider = new NodeTracerProvider(); provider.register(); @@ -249,7 +249,6 @@ describe('NodeTracerProvider', () => { 'x-b3-sampled', 'x-b3-parentspanid', 'uber-trace-id', - 'x-amzn-trace-id', ]); }); }); diff --git a/packages/opentelemetry-sdk-trace-node/tsconfig.json b/packages/opentelemetry-sdk-trace-node/tsconfig.json index a40ddeff365..31559ba9b3b 100644 --- a/packages/opentelemetry-sdk-trace-node/tsconfig.json +++ b/packages/opentelemetry-sdk-trace-node/tsconfig.json @@ -32,9 +32,6 @@ }, { "path": "../opentelemetry-semantic-conventions" - }, - { - "path": "../propagator-aws-xray" } ] }