Sourcemaps with @sentry/next broken

[badtant]

Hi, I have issues with "filename" that is reported with exceptions. Example with correct path from sentry/next 7.17.1

{
  "exception": {
    "values": [
      {
        "type": "CombinedError",
        "value": "[Network] Failed to fetch",
        "stacktrace": {
          "frames": [
            {
              "filename": "https://www.svtstatic.se/play/play7/_next/static/chunks/pages/_app-090ea98a43a3e541.js",
              "function": "?",
              "in_app": true,
              "lineno": 83,
              "colno": 4226
            },
            {
              "filename": "https://www.svtstatic.se/play/play7/_next/static/chunks/pages/_app-090ea98a43a3e541.js",
              "function": "tZ",
              "in_app": true,
              "lineno": 83,
              "colno": 1378
            }
          ]
        },
        "mechanism": {
          "type": "generic",
          "handled": true
        }
      }
    ]
  }

When upgrading sentry/next 7.56.0 we instead get this:

{
  "exception": {
    "values": [
      {
        "type": "CombinedError",
        "value": "[Network] Failed to fetch",
        "stacktrace": {
          "frames": [
            {
              "filename": "app:///_next/static/chunks/pages/_app-13b08673ff5a8a80.js",
              "function": "?",
              "in_app": true,
              "lineno": 83,
              "colno": 4226
            },
            {
              "filename": "app:///_next/static/chunks/pages/_app-13b08673ff5a8a80.js",
              "function": "tZ",
              "in_app": true,
              "lineno": 83,
              "colno": 1378
            }
          ]
        },
        "mechanism": {
          "type": "generic",
          "handled": true
        }
      }
    ]
  }

As you can see the filename is different and wont match the sourcemaps that we have uploaded to sentry.

My guess is that it's this change that got introduced in 7.24.0 #6388
We do have a next app that uses assetPrefix. Should I change my configuration to sentry in any way or it this a bug?

[lforst]

Hey, #6388 is a fix to the behaviour you're describing. Can you share what the artifacts uploaded inside the release look like?

[badtant]

removing urlPrefix results in the artifacts below and didn't work with "filename": "app:///_next/static/chunks/pages/_app-02d4f7426f5d8bdc.js"

~/_next/pages/_app-02d4f7426f5d8bdc.js
~/_next/pages/_app-02d4f7426f5d8bdc.js.map

now i'm gonna try this:

include: [
        {
            paths: ['.next/static/chunks'],
            urlPrefix: '~/_next/static/chunks',
        },
],

[lforst]

do you mean remove include totally or just remove urlPrefix?

Removing it entirely!

Can you share your next.config.js and SDK version? I have a feeling we're still setting some options that we should rather leave alone.

[badtant]

"@sentry/cli": "2.19.1",
"@sentry/nextjs": "7.56.0",

const SentryWebpackPluginOptions = {
    authToken: process.env.SENTRY_AUTH_TOKEN,
    org: 'xxx',
    project: 'yyy',
    include: [
        {
            paths: ['.next/static/chunks'],
            urlPrefix: '~/_next/static/chunks',
        },
    ],
    ignore: [],
    silent: true,
    dryRun: process.env.BRANCH !== 'master',
    release: sentryRelease,
};

[lforst]

Please try the following:

const SentryWebpackPluginOptions = {
    authToken: process.env.SENTRY_AUTH_TOKEN,
    org: 'xxx',
    project: 'yyy',
    silent: true,
    dryRun: process.env.BRANCH !== 'master',
    release: sentryRelease,
};

[badtant]

update:
this conf seems to work

include: [
        {
            paths: ['.next/static/chunks'],
            urlPrefix: '~/_next/static/chunks',
        },
],

now i'm gonna try to remove include and ignore entirely (your suggestion)

[badtant]

removing include and ignore works, thanks for all your help!

[simon-abbott]

If anyone is stumbling across this discussion trying to fix your stacktraces looking like this: app:///_next/static/chunks/968-123c3396992648da.js (note the lack of /pages/ in that path, that's important), try setting widenClientFileUpload: true in next.config.js.