#137

- preset/node: правки настройки sentry
sima-land · Jul 4, 2024 · 9ed268b · 9ed268b
1 parent 1cb5072
commit 9ed268b
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/src/preset/node/providers/express-request-middleware.ts b/src/preset/node/providers/express-request-middleware.ts
@@ -1,11 +1,12 @@
-import { withIsolationScope } from '@sentry/browser';
+import { withIsolationScope } from '@sentry/node';
 import type express from 'express';
 
 /**
  * Провайдер промежуточного слоя учета входящих http-запросов.
  * @return Промежуточный слой.
  */
 export function provideExpressRequestMiddleware(): express.Handler {
+  // https://github.com/getsentry/sentry-javascript/discussions/9618#discussioncomment-9950984
   return (req, res, next) => {
     withIsolationScope(() => {
       next();

diff --git a/src/preset/node/providers/log-handler-sentry.ts b/src/preset/node/providers/log-handler-sentry.ts
@@ -20,7 +20,9 @@ export function provideLogHandlerSentry(resolve: Resolve): LogHandler {
     tracesSampleRate: Number(source.get('SENTRY_TRACES_SAMPLE_RATE', 0)),
     profilesSampleRate: Number(source.get('SENTRY_TRACES_SAMPLE_RATE', 0)),
     integrations: [...getDefaultIntegrations({})],
-    skipOpenTelemetrySetup: true,
+
+    // ВАЖНО: данная опция ломает группировку ошибок, активировать при необэодимости связать компоненты OTEL с Sentry
+    // skipOpenTelemetrySetup: true,
   });
 
   // ВАЖНО: передаем функцию чтобы брать текущий scope в момент вызова метода logger'а

diff --git a/src/preset/node/providers/main-express-app.ts b/src/preset/node/providers/main-express-app.ts
@@ -55,9 +55,7 @@ export function provideMainExpressApp(resolve: Resolve): express.Application {
     const path = typeof routePath === 'string' ? routePath : routePath.path;
     const method = typeof routePath === 'string' ? 'get' : routePath.method;
 
-    app.use(path, middleware);
-    app[method](path, routeHandler);
-    app.use(path, endMiddleware);
+    app[method](path, middleware, routeHandler, endMiddleware);
   }
 
   // служебные маршруты (к ним не применяются промежуточные слои)