use window vars for sentry and correct env name (airbytehq#6802)

* fall back to window var for sentry initialization * correct naming * add dsn to window type * fix naming
khozzy · Oct 8, 2021 · 5d9d36b · 5d9d36b
1 parent ec68f47
commit 5d9d36b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/airbyte-webapp/.env b/airbyte-webapp/.env
@@ -2,5 +2,5 @@ REACT_APP_SEGMENT_TOKEN=6cxNSmQyGSKcATLdJ2pL6WsawkzEMDAN
 REACT_APP_FULL_STORY_ORG=13AXQ4
 REACT_APP_PAPERCUPS_ACCOUNT_ID=74560291-451e-4ceb-a802-56706ece528b
 REACT_APP_OPEN_REPLAY_PROJECT_ID=6611843272536134
-REACT_APP_SENTRY_DNS=
+REACT_APP_SENTRY_DSN=
 REACT_APP_INTERCOM_APP_ID=nj1oam7s
diff --git a/airbyte-webapp/src/config/types.ts b/airbyte-webapp/src/config/types.ts
@@ -17,6 +17,7 @@ declare global {
     FIREBASE_API_KEY?: string;
     FIREBASE_AUTH_DOMAIN?: string;
     CLOUD_API_URL?: string;
+    REACT_APP_SENTRY_DSN?: string;
 
     analytics: SegmentAnalytics;
     _API_URL: string;

diff --git a/airbyte-webapp/src/index.tsx b/airbyte-webapp/src/index.tsx
@@ -4,7 +4,9 @@ import * as Sentry from "@sentry/react";
 
 // We do not follow default config approach that we follow as we want to init
 // sentry asap
-Sentry.init({ dsn: process.env.REACT_APP_SENTRY_DNS });
+Sentry.init({
+  dsn: process.env.REACT_APP_SENTRY_DSN || window.REACT_APP_SENTRY_DSN,
+});
 
 const CloudApp = lazy(() => import(`packages/cloud/App`));
 const App = lazy(() => import(`./App`));