fix: sentry user data now augments correctly (#136) (#147)

* fix: sentry user data now augments correctly (#136) * chore: use environment * chore: update environment to use vercel env where applicable
Kimberley-College · Mar 19, 2022 · 242545d · 242545d
1 parent 8b31739
commit 242545d
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/sentry.client.config.js b/sentry.client.config.js
@@ -14,4 +14,5 @@ Sentry.init({
   // Note: if you want to override the automatic release value, do not set a
   // `release` value here - use the environment variable `SENTRY_RELEASE`, so
   // that it will also get attached to your source maps
+  environment: process.env.VERCEL_ENV ?? process.env.NODE_ENV,
 });
diff --git a/sentry.server.config.js b/sentry.server.config.js
@@ -14,4 +14,5 @@ Sentry.init({
   // Note: if you want to override the automatic release value, do not set a
   // `release` value here - use the environment variable `SENTRY_RELEASE`, so
   // that it will also get attached to your source maps
+  environment: process.env.VERCEL_ENV ?? process.env.NODE_ENV,
 });
diff --git a/src/pages/_app.tsx b/src/pages/_app.tsx
@@ -13,9 +13,11 @@ import theme from '../util/theme';
 const App: NextPage<AppProps> = ({ Component, pageProps }) => {
   useAnalytics({ publicKey: process.env.NEXT_PUBLIC_HAPPYKIT_ANALYTICS_PUBLIC_KEY });
   useEffect(() => {
-    const { data: authListener } = supabaseClient.auth.onAuthStateChange(async (_, session) => {
-      if (!session) Sentry.setUser(null);
-      else {
+    console.log('effect');
+    const { data: authListener } = supabaseClient.auth.onAuthStateChange((event, session) => {
+      if (event === 'SIGNED_OUT') {
+        Sentry.setUser(null);
+      } else if (session) {
         Sentry.setUser({
           email: session.user.email,
           username: session.user.user_metadata.proper_name,