Stacktraces not source mapped after upgrading from 7.26 to 7.42 (Next.js assetPrefix) #7500

Activity

1. eirikur-grid

   added

   Type: Bug

   on Mar 17, 2023
2. getsantry

   added

   Status: Untriaged

   on Mar 17, 2023
3. Meemaw commented on Mar 17, 2023

   

   Meemaw

   on Mar 17, 2023

   Last edited by Meemaw

   More actions

   Having similar issues as well.
4. Lms24

   added

   Package: Nextjs

   on Mar 20, 2023
5. getsantry

   removed

   Status: Untriaged

   on Mar 20, 2023
6. lforst commented on Mar 20, 2023

   

   lforst

   on Mar 20, 2023

   Last edited by lforst

   Contributor

   More actions

   Hi, can you share your next.config.js? thanks!
7. eirikur-grid commented on Mar 20, 2023

   

   eirikur-grid

   on Mar 20, 2023

   Author

   More actions

   Here it is

   /* globals module require process __dirname */
   const path = require('path');
   const { DefinePlugin, IgnorePlugin } = require('webpack');
   const { withSentryConfig } = require('@sentry/nextjs');
   
   // As an exception to the general rule of not transpiling anything under node_modules
   // we transpile these modules that do not provide the same level of browser support.
   const withTM = require('next-transpile-modules')([
     '@grid-is/apiary',
     '@borgar/fx',
     'antlr4',
   ]);
   
   let gitHash;
   /* git hash determination omitted */
   
   let gitBranch;
   /* git branch determination omitted */
   
   const runDate = new Date()
     .toDateString()
     .split(' ')
     .slice(1, 3)
     .join('-')
     .toLowerCase();
   const buildId = runDate + ':' + gitHash;
   
   const assetPrefix = process.env.ASSET_PREFIX || '';
   
   const sendSourceMaps =
     (gitBranch === 'develop' || gitBranch === 'master');
     /* other conditions omitted */
   
   const SentryWebpackPluginOptions = {
     release: buildId,
     include: '.next',
     setCommits: {
       auto: true,
     },
     silent: false,
     // For all available options, see:
     // https://github.com/getsentry/sentry-webpack-plugin#options.
   };
   const moduleExports = {
     // strict mode is a development mode only feature for highlighting potential problems in an application
     // see https://nextjs.org/docs/api-reference/next.config.js/react-strict-mode for more info
     // reactStrictMode: true,
     sentry: {
       disableServerWebpackPlugin: !sendSourceMaps,
       disableClientWebpackPlugin: !sendSourceMaps,
     },
     outputFileTracing: false, // workaround for https://github.com/getsentry/sentry-javascript/issues/4103
     serverRuntimeConfig: {
       rootDir: __dirname,
       dynamicRendering: process.env.DYNAMIC_RENDERING,
     },
     sassOptions: {
       includePaths: [ path.join(__dirname, 'src', 'styles') ],
     },
     experimental: {
       esmExternals: 'loose',
     },
     async rewrites () {
       return [
         /* omitted */
       ];
     },
     async redirects () {
       return [
         /* omitted */
       ];
     },
     async headers () {
       return [
         {
           source: '/_next/static/wasm/:all*',
           locale: false,
           headers: [
             {
               key: 'Cache-Control',
               value: 'public, max-age=31536001, immutable',
             },
           ],
         },
       ];
     },
     webpack: (config, { isServer, dev }) => {
       config.experiments.asyncWebAssembly = true;
       config.resolve.fallback = {
         ...config.resolve.fallback,
         fs: false,
       };
   
       // Be default, font filenames will have a hash appended to them.
       // That can make it tricky to reference them directly, e.g. for
       // preloading, because the URL might not be stable.
       // The following rule is intended to suppress that behavior.
       config.module.rules.push({
         test: /\.(woff(2)?|ttf|eot)$/,
         type: 'asset/resource',
         generator: {
           filename: './static/media/fonts/[name][ext]',
         },
       });
   
       let sentryEnv;
       // HACK: presume that development is exclusive to computers running MacOS
       // XXX: read the sentry environment from an environment variable
       if (dev || process.platform === 'darwin') {
         sentryEnv = 'localdev';
       }
       else if (gitBranch === 'master') {
         sentryEnv = 'production';
       }
       else if (gitBranch === 'develop') {
         sentryEnv = 'staging';
       }
       else {
         sentryEnv = 'branch-deployment';
       }
       // Workaround https://github.com/oguimbal/pg-mem/issues/225
       config.plugins.push(
         new IgnorePlugin({
           resourceRegExp: /^@mikro-orm\/|^knex|typeorm|slonik|pg-promise/,
           contextRegExp: /pg-mem/,
         }),
       );
       config.plugins.push(
         new DefinePlugin({
           SENTRY_ENV: `"${sentryEnv}"`,
           /* others redacted */
         }),
       );
   
       return config;
     },
   };
   
   if (assetPrefix) {
     moduleExports.assetPrefix = assetPrefix;
   }
   
   /**
    * @param {string} phase
    * @param {{ defaultConfig: import("@sentry/nextjs/types/config/types").NextConfigObject; }} defaults
    * @returns {import('@sentry/nextjs/types/config/types').NextConfigObject}
    */
   module.exports = (phase, defaults) => {
     const sentryConfig = withSentryConfig(
       moduleExports,
       SentryWebpackPluginOptions,
     );
   
     // Here sentryConfig is either a config object or a function that returns one.
     // If it is a function, we must call that function, because the subsequent
     // config modifiers `withTM` and `withBundleAnalyzer` both assume they get a
     // config object, not a function (and if they do get a function, they don't
     // notice, and just treat it like a blank configuration object, yielding
     // oh-so-confusing build failures.)
   
     /** @type {import('next').NextConfig} */
     let config;
     if (typeof sentryConfig === 'function') {
       config = sentryConfig(phase, defaults);
     }
     else {
       config = sentryConfig;
     }
   
     config = withTM(config);
   
     if (process.env.ANALYZE === 'true') {
       const withBundleAnalyzer = require('@next/bundle-analyzer')({
         enabled: true,
       });
       config = withBundleAnalyzer(config);
     }
   
     return config;
   };
8. lforst commented on Mar 27, 2023

   

   lforst

   on Mar 27, 2023

   Contributor

   More actions

   Thanks, this was a regression and we have identified the cause. A fix is on its way.

   👍React with 👍1Reacted by Eiríkur Fannar Torfason
9. lforst

   closed this as completedin #7619on Mar 27, 2023