Merge pull request coderoad#203 from ShMcK/cleanup-env

remove sentry DSN

Author: ShMcK

src/environment.ts

@@ -7,13 +7,15 @@ interface Environment {
   NODE_ENV: string
   LOG: boolean
   API_URL: string
+  SENTRY_DSN: string | null
 }
 
 const environment: Environment = {
   VERSION: process.env.VERSION || 'unknown',
   NODE_ENV: process.env.NODE_ENV || 'production',
   LOG: (process.env.LOG || '').toLowerCase() === 'true',
   API_URL: process.env.REACT_APP_GQL_URI || '',
+  SENTRY_DSN: process.env.SENTRY_DSN || null,
 }
 
 export default environment

src/services/sentry/init.ts

@@ -1,7 +1,9 @@
 import { init } from '@sentry/node'
 import environment from '../../environment'
 
-init({
-  dsn: 'https://df4a6ae19e8b44ed9a87ae4432dab9df@sentry.io/1889368',
-  environment: environment.NODE_ENV,
-})
+if (environment.SENTRY_DSN) {
+  init({
+    dsn: environment.SENTRY_DSN,
+    environment: environment.NODE_ENV,
+  })
+}

web-app/src/environment.ts

@@ -11,3 +11,4 @@ export const VERSION: string = process.env.VERSION || 'unknown'
 export const NODE_ENV: string = process.env.NODE_ENV || 'development'
 export const LOG_STATE: boolean = (process.env.REACT_APP_LOG_STATE || '').toLowerCase() === 'true'
 export const TUTORIAL_LIST_URL: string = process.env.REACT_APP_TUTORIAL_LIST_URL || ''
+export const SENTRY_DSN: string | null = process.env.REACT_APP_SENTRY_DSN || null

web-app/src/services/sentry/init.tsx

@@ -1,11 +1,13 @@
 import * as sentry from '@sentry/browser'
-import { NODE_ENV } from '../../environment'
+import { NODE_ENV, SENTRY_DSN } from '../../environment'
 
 try {
-  sentry.init({
-    dsn: 'https://701cee76c32a4408b2fcb6af3e139d46@sentry.io/1889371',
-    environment: NODE_ENV,
-  })
+  if (SENTRY_DSN) {
+    sentry.init({
+      dsn: SENTRY_DSN,
+      environment: NODE_ENV,
+    })
+  }
 } catch (error) {
   console.log('Error in Sentry init')
   console.log(error)