fix(sentry): correctly setups the sentry integration

fixes #92

.gitignore

@@ -57,3 +57,6 @@ build-storybook.log
 .sentryclirc
 
 types/nexus-typegen
+
+# Sentry
+.sentryclirc

next.config.js

@@ -1,3 +1,15 @@
-module.exports = {
+// eslint-disable-next-line @typescript-eslint/no-var-requires
+const { withSentryConfig } = require('@sentry/nextjs');
+
+const moduleExports = {
   reactStrictMode: true,
+  sentry: {
+    hideSourceMaps: false,
+  },
 };
+
+const sentryWebpackPluginOptions = {
+  silent: true,
+};
+
+module.exports = withSentryConfig(moduleExports, sentryWebpackPluginOptions);

package.json

@@ -33,7 +33,7 @@
     "@hookform/resolvers": "^2.9.11",
     "@next-auth/prisma-adapter": "^1.0.5",
     "@prisma/client": "^4.10.1",
-    "@sentry/nextjs": "^7.37.2",
+    "@sentry/nextjs": "^7.38.0",
     "@types/bytes": "^3.1.1",
     "@types/graphql-upload": "^15.0.2",
     "apollo-server-micro": "^3.11.1",

pages/_error.js

@@ -14,16 +14,20 @@ const MyError = ({ statusCode, hasGetInitialPropsRun, err }) => {
   return <NextErrorComponent statusCode={statusCode} />;
 };
 
-MyError.getInitialProps = async ({ res, err, asPath }) => {
-  const errorInitialProps = await NextErrorComponent.getInitialProps({
-    res,
-    err,
-  });
+MyError.getInitialProps = async (context) => {
+  const errorInitialProps = await NextErrorComponent.getInitialProps(context);
+
+  const { res, err, asPath } = context;
 
   // Workaround for https://github.com/vercel/next.js/issues/8592, mark when
   // getInitialProps has run
   errorInitialProps.hasGetInitialPropsRun = true;
 
+  // Returning early because we don't want to log 404 errors to Sentry.
+  if (res?.statusCode === 404) {
+    return errorInitialProps;
+  }
+
   // Running on the server, the response object (`res`) is available.
   //
   // Next.js will pass an err on the server if a page's data fetching methods
@@ -50,9 +54,7 @@ MyError.getInitialProps = async ({ res, err, asPath }) => {
   // If this point is reached, getInitialProps was called without any
   // information about what the error might be. This is unexpected and may
   // indicate a bug introduced in Next.js, so record it in Sentry
-  Sentry.captureException(
-    new Error(`_error.js getInitialProps missing data at path: ${asPath}`),
-  );
+  Sentry.captureException(new Error(`_error.js getInitialProps missing data at path: ${asPath}`));
   await Sentry.flush(2000);
 
   return errorInitialProps;

sentry.client.config.js

@@ -10,6 +10,7 @@ Sentry.init({
   dsn: SENTRY_DSN || 'https://4e506b139137430fa20d43cd2d526912@o1080632.ingest.sentry.io/6086695',
   // Adjust this value in production, or use tracesSampler for greater control
   tracesSampleRate: 1.0,
+  environment: process.env.NEXT_PUBLIC_ENVIRONMENT || 'development',
   // ...
   // Note: if you want to override the automatic release value, do not set a
   // `release` value here - use the environment variable `SENTRY_RELEASE`, so

sentry.server.config.js

@@ -10,6 +10,7 @@ Sentry.init({
   dsn: SENTRY_DSN || 'https://4e506b139137430fa20d43cd2d526912@o1080632.ingest.sentry.io/6086695',
   // Adjust this value in production, or use tracesSampler for greater control
   tracesSampleRate: 1.0,
+  environment: process.env.NEXT_PUBLIC_ENVIRONMENT || 'development',
   // ...
   // Note: if you want to override the automatic release value, do not set a
   // `release` value here - use the environment variable `SENTRY_RELEASE`, so