Skip to content

Commit 9f25f58

Browse files
authored
Decouple SigV4 from Application Signals (#176)
*Description of changes:* We do not want to automatically disable Application Signals if SigV4 is enabled. This PR allows Application Signals to run normally with SigV4. By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
1 parent 89a9b2b commit 9f25f58

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

aws-distro-opentelemetry-node-autoinstrumentation/src/aws-opentelemetry-configurator.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -240,10 +240,6 @@ export class AwsOpentelemetryConfigurator {
240240

241241
spanProcessors.push(AttributePropagatingSpanProcessorBuilder.create().build());
242242

243-
if (isXrayOtlpEndpoint(process.env['OTEL_EXPORTER_OTLP_TRACES_ENDPOINT'])) {
244-
return;
245-
}
246-
247243
const applicationSignalsMetricExporter: PushMetricExporter =
248244
ApplicationSignalsExporterProvider.Instance.createExporter();
249245
const periodicExportingMetricReader: PeriodicExportingMetricReader = new PeriodicExportingMetricReader({

0 commit comments

Comments
 (0)