Do not set NoopSpanProcessor if no span processor is set

myrotvorets · Sep 30, 2023 · 461219b · 461219b
1 parent d2d0f7a
commit 461219b
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/lib/index.mts b/lib/index.mts
@@ -1,6 +1,6 @@
 import { processDetectorSync } from '@opentelemetry/resources';
 import { NodeSDK, type NodeSDKConfiguration } from '@opentelemetry/sdk-node';
-import { BatchSpanProcessor, NoopSpanProcessor, SimpleSpanProcessor } from '@opentelemetry/sdk-trace-base';
+import { BatchSpanProcessor, SimpleSpanProcessor } from '@opentelemetry/sdk-trace-base';
 import { ZipkinExporter } from '@opentelemetry/exporter-zipkin';
 import {
     dockerDetector,
@@ -28,8 +28,6 @@ export class OpenTelemetryConfigurator {
                         ? new BatchSpanProcessor(this._config.traceExporter)
                         : new SimpleSpanProcessor(this._config.traceExporter);
                 /* c8 disable end */
-            } else {
-                this._config.spanProcessor = new NoopSpanProcessor();
             }
         }