Ver Fonte

fix: remove sentry for community edtion and dev (#259)

Joel há 1 ano atrás
pai
commit
54bb309d87
1 ficheiros alterados com 13 adições e 5 exclusões
  1. 13 5
      web/next.config.js

+ 13 - 5
web/next.config.js

@@ -1,5 +1,10 @@
 const { withSentryConfig } = require('@sentry/nextjs')
 
+const EDITION = process.env.NEXT_PUBLIC_EDITION
+const IS_CE_EDITION = EDITION === 'SELF_HOSTED'
+const isDevelopment = process.env.NODE_ENV === 'development'
+const isHideSentry = isDevelopment || IS_CE_EDITION
+
 const withMDX = require('@next/mdx')({
   extension: /\.mdx?$/,
   options: {
@@ -31,9 +36,6 @@ const nextConfig = {
     // https://nextjs.org/docs/api-reference/next.config.js/ignoring-typescript-errors
     ignoreBuildErrors: true,
   },
-  sentry: {
-    hideSourceMaps: true,
-  },
   async redirects() {
     return [
       {
@@ -43,6 +45,13 @@ const nextConfig = {
       },
     ]
   },
+  ...(isHideSentry
+    ? {}
+    : {
+      sentry: {
+        hideSourceMaps: true,
+      },
+    }),
 }
 
 // https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup
@@ -54,8 +63,7 @@ const sentryWebpackPluginOptions = {
     assets: './**',
     ignore: ['./node_modules/**'],
   },
-
   // https://github.com/getsentry/sentry-webpack-plugin#options.
 }
 
-module.exports = withMDX(withSentryConfig(nextConfig, sentryWebpackPluginOptions))
+module.exports = isHideSentry ? withMDX(nextConfig) : withMDX(withSentryConfig(nextConfig, sentryWebpackPluginOptions))