From fab68679f5aa67d87c3b94b7132f21d53940e13c Mon Sep 17 00:00:00 2001
From: Alisha Evans <alishaevn2@gmail.com>
Date: Tue, 22 Aug 2023 12:55:11 -0500
Subject: [PATCH] conditional sentry exception handling

clients ought to have the option to use whichever exception handler they want, not only sentry. by default, the apps that are set up by softserv will use sentry however. this change allows the app to still build whether the sentry values in .env.local are set or not.
whether that means the app will use something other than sentry, or the sentry project has not been set up yet
---
 next.config.js | 84 ++++++++++++++++++++++++++------------------------
 1 file changed, 43 insertions(+), 41 deletions(-)

diff --git a/next.config.js b/next.config.js
index b8836c15..ed2858fa 100644
--- a/next.config.js
+++ b/next.config.js
@@ -1,45 +1,47 @@
-// This file sets a custom webpack configuration to use your Next.js app
-// with Sentry.
-// https://nextjs.org/docs/api-reference/next.config.js/introduction
-// https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/
-const { withSentryConfig } = require('@sentry/nextjs')
+if (process.env.SENTRY_DSN || process.env.NEXT_PUBLIC_SENTRY_DSN) {
+  // This file sets a custom webpack configuration to use your Next.js app
+  // with Sentry.
+  // https://nextjs.org/docs/api-reference/next.config.js/introduction
+  // https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/
+  const { withSentryConfig } = require('@sentry/nextjs')
 
-/** @type {import('next').NextConfig} */
-const moduleExports = {
-  output: 'standalone',
-  reactStrictMode: true,
-  swcMinify: true,
-  eslint: {
-    dirs: ['pages', 'utils'], // Only run ESLint on the 'pages' and 'utils' directories during production builds
-  },
-  sentry: {
-    // Use `hidden-source-map` rather than `source-map` as the Webpack `devtool`
-    // for client-side builds. (This will be the default starting in
-    // `@sentry/nextjs` version 8.0.0.) See
-    // https://webpack.js.org/configuration/devtool/ and
-    // https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/#use-hidden-source-map
-    // for more information.
-    hideSourceMaps: true,
-  },
-}
+  /** @type {import('next').NextConfig} */
+  const moduleExports = {
+    output: 'standalone',
+    reactStrictMode: true,
+    swcMinify: true,
+    eslint: {
+      dirs: ['pages', 'utils'], // Only run ESLint on the 'pages' and 'utils' directories during production builds
+    },
+    sentry: {
+      // Use `hidden-source-map` rather than `source-map` as the Webpack `devtool`
+      // for client-side builds. (This will be the default starting in
+      // `@sentry/nextjs` version 8.0.0.) See
+      // https://webpack.js.org/configuration/devtool/ and
+      // https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/#use-hidden-source-map
+      // for more information.
+      hideSourceMaps: true,
+    },
+  }
 
-const sentryWebpackPluginOptions = {
-  // Additional config options for the Sentry Webpack plugin. Keep in mind that
-  // the following options are set automatically, and overriding them is not
-  // recommended:
-  //   release, url, configFile, stripPrefix, urlPrefix, include, ignore
+  const sentryWebpackPluginOptions = {
+    // Additional config options for the Sentry Webpack plugin. Keep in mind that
+    // the following options are set automatically, and overriding them is not
+    // recommended:
+    //   release, url, configFile, stripPrefix, urlPrefix, include, ignore
 
-  silent: true, // Suppresses all logs
-  project: process.env.SENTRY_PROJECT,
-  org: process.env.SENTRY_ORG,
-  authToken: process.env.SENTRY_AUTH_TOKEN,
-  // For all available options, see:
-  // https://github.com/getsentry/sentry-webpack-plugin#options.
-}
+    silent: true, // Suppresses all logs
+    project: process.env.SENTRY_PROJECT,
+    org: process.env.SENTRY_ORG,
+    authToken: process.env.SENTRY_AUTH_TOKEN,
+    // For all available options, see:
+    // https://github.com/getsentry/sentry-webpack-plugin#options.
+  }
 
-// Make sure adding Sentry options is the last code to run before exporting, to
-// ensure that your source maps include changes from all other Webpack plugins
-module.exports = withSentryConfig(
-  moduleExports,
-  sentryWebpackPluginOptions
-)
+  // Make sure adding Sentry options is the last code to run before exporting, to
+  // ensure that your source maps include changes from all other Webpack plugins
+  module.exports = withSentryConfig(
+    moduleExports,
+    sentryWebpackPluginOptions
+  )
+}