fix: rename sentry config property from DNS to DSN

Aam-Digital · Feb 29, 2024 · ad1c2de · ad1c2de
1 parent 076a5b3
commit ad1c2de
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/config/app.yaml b/src/config/app.yaml
@@ -32,4 +32,4 @@ SENTRY:
   ENABLED: false
   INSTANCE_NAME: local-development # can be personalised in .env -> local-development-<your-name>
   ENVIRONMENT: local # local | development | production
-  DNS: ''
+  DSN: ''
diff --git a/src/sentry.configuration.ts b/src/sentry.configuration.ts
@@ -5,7 +5,7 @@ import { BaseExceptionFilter, HttpAdapterHost } from '@nestjs/core';
 
 export class SentryConfiguration {
   ENABLED = '';
-  DNS = '';
+  DSN = '';
   INSTANCE_NAME = '';
   ENVIRONMENT = '';
 }
@@ -15,7 +15,7 @@ function loadSentryConfiguration(
 ): SentryConfiguration {
   return {
     ENABLED: configService.getOrThrow('SENTRY_ENABLED'),
-    DNS: configService.getOrThrow('SENTRY_DNS'),
+    DSN: configService.getOrThrow('SENTRY_DSN'),
     INSTANCE_NAME: configService.getOrThrow('SENTRY_INSTANCE_NAME'),
     ENVIRONMENT: configService.getOrThrow('SENTRY_ENVIRONMENT'),
   };
@@ -46,7 +46,7 @@ function configureLoggingSentry(
     debug: true,
     serverName: sentryConfiguration.INSTANCE_NAME,
     environment: sentryConfiguration.ENVIRONMENT,
-    dsn: sentryConfiguration.DNS,
+    dsn: sentryConfiguration.DSN,
     integrations: [
       // enable HTTP calls tracing
       new Sentry.Integrations.Console(),