chore: Update la gestion du scope pour sentry v8

DNUM-SocialGouv · May 16, 2024 · 6b8b314 · 6b8b314
1 parent 5e4cbdd
commit 6b8b314
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/client/services/bff.httpClient.service.ts b/src/client/services/bff.httpClient.service.ts
@@ -22,6 +22,8 @@ export class BffHttpClientService implements HttpClientService {
 			(request: InternalAxiosRequestConfig) => {
 				const transactionId = uuid4();
 				request.headers['x-transaction-id'] = transactionId;
+				// FIXME (GAFI 16-05-2024): On est d'accord que là, si j'ai 2 requêtes qui partent coup sur coup, le transactionId
+				//	de la première est écrasé par la seconde ?
 				this.logger.setTransactionId(transactionId);
 				return request;
 			},

diff --git a/src/client/services/logger.service.ts b/src/client/services/logger.service.ts
@@ -2,9 +2,8 @@ import * as Sentry from '@sentry/nextjs';
 
 export class LoggerService {
 	constructor(sessionId?: string) {
-		Sentry.configureScope((scope: Sentry.Scope) => {
-			scope.setTag('session_id', sessionId);
-		});
+		const scope = Sentry.getCurrentScope();
+		scope.setTag('session_id', sessionId);
 	}
 
 	private static log(
@@ -27,8 +26,7 @@ export class LoggerService {
 	}
 
 	setTransactionId(transactionId: string): void {
-		Sentry.configureScope((scope) => {
-			scope.setTag('transaction_id', transactionId);
-		});
+		const scope = Sentry.getCurrentScope();
+		scope.setTag('transaction_id', transactionId);
 	}
 }