fix: Rename VITE_ENVIRONMENT to VITE_SENTRY_ENVIRONMENT for consistency in Sentry configuration

added try catch for url parsing

Author: n3rdc4ptn

.env.template

@@ -1,4 +1,4 @@
-VITE_ENVIRONMENT=
+VITE_SENTRY_ENVIRONMENT=
 # Sentry DSN configuration, no need to be set for local setup
 #  Frontend:
 SENTRY_ORG=

server.js

@@ -19,7 +19,7 @@ if (!process.env.BFF_SENTRY_DSN || process.env.BFF_SENTRY_DSN.trim() === '') {
     // Setting this option to true will send default PII data to Sentry.
     // For example, automatic IP address collection on events
     sendDefaultPii: true,
-    environment: process.env.VITE_ENVIRONMENT,
+    environment: process.env.VITE_SENTRY_ENVIRONMENT,
     beforeSend(event) {
       if (event.request && event.request.cookies) {
         event.request.cookies = Object.keys(event.request.cookies).reduce((acc, key) => {
@@ -51,9 +51,14 @@ const fastify = Fastify({
 Sentry.setupFastifyErrorHandler(fastify);
 await fastify.register(envPlugin);
 
-let sentryHost = null;
+let sentryHost = '';
 if (fastify.config.BFF_SENTRY_DSN && fastify.config.BFF_SENTRY_DSN.length > 0) {
-  sentryHost = new URL(fastify.config.BFF_SENTRY_DSN).hostname;
+  try {
+    sentryHost = new URL(fastify.config.BFF_SENTRY_DSN).hostname;
+  } catch {
+    console.log('BFF_SENTRY_DSN is not a valid URL');
+    sentryHost = '';
+  }
 }
 
 fastify.register(helmet, {

server/config/env.js

@@ -31,7 +31,7 @@ const schema = {
     FRAME_ANCESTORS: { type: 'string' },
     BFF_SENTRY_DSN: { type: 'string' },
     VITE_SENTRY_DSN: { type: 'string' },
-    VITE_ENVIRONMENT: { type: 'string' },
+    VITE_SENTRY_ENVIRONMENT: { type: 'string' },
 
     // System variables
     NODE_ENV: { type: 'string', enum: ['development', 'production'] },

src/mount.ts

@@ -11,7 +11,7 @@ if (import.meta.env.VITE_SENTRY_DSN && import.meta.env.VITE_SENTRY_DSN.length >
     // Setting this option to true will send default PII data to Sentry.
     // For example, automatic IP address collection on events
     sendDefaultPii: true,
-    environment: import.meta.env.VITE_ENVIRONMENT,
+    environment: import.meta.env.VITE_SENTRY_ENVIRONMENT,
     integrations: [
       Sentry.reactRouterV7BrowserTracingIntegration({
         useEffect: React.useEffect,