Avoid trying to launch sentry for instances without Sentry DSN key

GeotrekCE · Nov 29, 2024 · 5f2ea2b · 5f2ea2b
1 parent 626a9eb
commit 5f2ea2b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/frontend/src/instrumentation.js → frontend/src/instrumentation.ts b/frontend/src/instrumentation.js → frontend/src/instrumentation.ts
@@ -1,5 +1,3 @@
-import * as Sentry from '@sentry/nextjs';
-
 export async function register() {
   if (process.env.NEXT_RUNTIME === 'nodejs') {
     await import('./sentry.server.config');

diff --git a/frontend/src/sentry.client.config.js → frontend/src/sentry.client.config.ts b/frontend/src/sentry.client.config.js → frontend/src/sentry.client.config.ts
@@ -9,12 +9,14 @@ const sentryOptions = {
   maxBreadcrumbs: 50,
   attachStacktrace: true,
   integrations: [Sentry.replayIntegration()],
-};
+} as Sentry.BrowserOptions;
 
 // Development & tests setup
 if (process.env.NODE_ENV !== 'production') {
   // Don't actually send the errors to Sentry
   sentryOptions.beforeSend = () => null;
 }
 
-Sentry.init(sentryOptions);
+if (SENTRY_DSN) {
+  Sentry.init(sentryOptions);
+}
diff --git a/frontend/src/sentry.server.config.js → frontend/src/sentry.server.config.ts b/frontend/src/sentry.server.config.js → frontend/src/sentry.server.config.ts
@@ -8,12 +8,14 @@ const sentryOptions = {
   environment: process.env.ENVIRONMENT,
   maxBreadcrumbs: 50,
   attachStacktrace: true,
-};
+} as Sentry.NodeOptions;
 
 // Development & tests setup
 if (process.env.NODE_ENV !== 'production') {
   // Don't actually send the errors to Sentry
   sentryOptions.beforeSend = () => null;
 }
 
-Sentry.init(sentryOptions);
+if (SENTRY_DSN) {
+  Sentry.init(sentryOptions);
+}