diff --git a/packages/auth/src/Bootstrap/Container.ts b/packages/auth/src/Bootstrap/Container.ts index ad8403733..df216df93 100644 --- a/packages/auth/src/Bootstrap/Container.ts +++ b/packages/auth/src/Bootstrap/Container.ts @@ -1246,11 +1246,7 @@ export class ContainerConfigLoader { .toConstantValue( isConfiguredForHomeServerOrSelfHosting ? new SQSEventMessageHandler(eventHandlers, container.get(TYPES.Auth_Logger)) - : new SQSOpenTelemetryEventMessageHandler( - ServiceIdentifier.NAMES.AuthWorker, - eventHandlers, - container.get(TYPES.Auth_Logger), - ), + : new SQSOpenTelemetryEventMessageHandler(eventHandlers, container.get(TYPES.Auth_Logger)), ) container diff --git a/packages/domain-events-infra/src/Infra/SQS/SQSOpenTelemetryEventMessageHandler.ts b/packages/domain-events-infra/src/Infra/SQS/SQSOpenTelemetryEventMessageHandler.ts index 0fcf21ef5..ae02fb3cd 100644 --- a/packages/domain-events-infra/src/Infra/SQS/SQSOpenTelemetryEventMessageHandler.ts +++ b/packages/domain-events-infra/src/Infra/SQS/SQSOpenTelemetryEventMessageHandler.ts @@ -9,10 +9,8 @@ import { export class SQSOpenTelemetryEventMessageHandler implements DomainEventMessageHandlerInterface { private currentSpan: OpenTelemetryApi.Span | undefined - private internalSpan: OpenTelemetryApi.Span | undefined constructor( - private serviceName: string, private handlers: Map, private logger: Logger, ) {} @@ -37,26 +35,18 @@ export class SQSOpenTelemetryEventMessageHandler implements DomainEventMessageHa const tracer = OpenTelemetryApi.trace.getTracer('sqs-handler') - this.currentSpan = tracer.startSpan(this.serviceName, { kind: OpenTelemetryApi.SpanKind.CONSUMER }) - this.internalSpan = tracer.startSpan(domainEvent.type, { kind: OpenTelemetryApi.SpanKind.INTERNAL }) + this.currentSpan = tracer.startSpan(domainEvent.type) await handler.handle(domainEvent) - this.internalSpan.end() this.currentSpan.end() - this.internalSpan = undefined this.currentSpan = undefined } async handleError(error: Error): Promise { - if (this.internalSpan) { - this.internalSpan.recordException(error) - this.internalSpan.end() - this.internalSpan = undefined - } - if (this.currentSpan) { + this.currentSpan.recordException(error) this.currentSpan.end() this.currentSpan = undefined } diff --git a/packages/files/src/Bootstrap/Container.ts b/packages/files/src/Bootstrap/Container.ts index 46edf17d7..f6edbc4ae 100644 --- a/packages/files/src/Bootstrap/Container.ts +++ b/packages/files/src/Bootstrap/Container.ts @@ -316,11 +316,7 @@ export class ContainerConfigLoader { .toConstantValue( isConfiguredForHomeServerOrSelfHosting ? new SQSEventMessageHandler(eventHandlers, container.get(TYPES.Files_Logger)) - : new SQSOpenTelemetryEventMessageHandler( - ServiceIdentifier.NAMES.FilesWorker, - eventHandlers, - container.get(TYPES.Files_Logger), - ), + : new SQSOpenTelemetryEventMessageHandler(eventHandlers, container.get(TYPES.Files_Logger)), ) container .bind(TYPES.Files_DomainEventSubscriberFactory) diff --git a/packages/revisions/src/Bootstrap/Container.ts b/packages/revisions/src/Bootstrap/Container.ts index 93605dce4..9121e29fb 100644 --- a/packages/revisions/src/Bootstrap/Container.ts +++ b/packages/revisions/src/Bootstrap/Container.ts @@ -533,11 +533,7 @@ export class ContainerConfigLoader { .toConstantValue( isConfiguredForHomeServerOrSelfHosting ? new SQSEventMessageHandler(eventHandlers, container.get(TYPES.Revisions_Logger)) - : new SQSOpenTelemetryEventMessageHandler( - ServiceIdentifier.NAMES.RevisionsWorker, - eventHandlers, - container.get(TYPES.Revisions_Logger), - ), + : new SQSOpenTelemetryEventMessageHandler(eventHandlers, container.get(TYPES.Revisions_Logger)), ) container diff --git a/packages/syncing-server/src/Bootstrap/Container.ts b/packages/syncing-server/src/Bootstrap/Container.ts index bb370748c..57142353e 100644 --- a/packages/syncing-server/src/Bootstrap/Container.ts +++ b/packages/syncing-server/src/Bootstrap/Container.ts @@ -1178,11 +1178,7 @@ export class ContainerConfigLoader { .toConstantValue( isConfiguredForHomeServerOrSelfHosting ? new SQSEventMessageHandler(eventHandlers, container.get(TYPES.Sync_Logger)) - : new SQSOpenTelemetryEventMessageHandler( - ServiceIdentifier.NAMES.SyncingServerWorker, - eventHandlers, - container.get(TYPES.Sync_Logger), - ), + : new SQSOpenTelemetryEventMessageHandler(eventHandlers, container.get(TYPES.Sync_Logger)), ) } diff --git a/packages/websockets/src/Bootstrap/Container.ts b/packages/websockets/src/Bootstrap/Container.ts index ea8f3420d..c093cc5c0 100644 --- a/packages/websockets/src/Bootstrap/Container.ts +++ b/packages/websockets/src/Bootstrap/Container.ts @@ -164,13 +164,7 @@ export class ContainerConfigLoader { container .bind(TYPES.DomainEventMessageHandler) - .toConstantValue( - new SQSOpenTelemetryEventMessageHandler( - ServiceIdentifier.NAMES.WebsocketsWorker, - eventHandlers, - container.get(TYPES.Logger), - ), - ) + .toConstantValue(new SQSOpenTelemetryEventMessageHandler(eventHandlers, container.get(TYPES.Logger))) container .bind(TYPES.DomainEventSubscriberFactory) .toConstantValue(