Bug: Stack trace link to URL picking up trailing parenthesis

[mwarkentin]

Environment

SaaS (https://sentry.io/)

Version

No response

Steps to Reproduce

Trigger an exception which includes a URL like:

Loading chunk app_bootstrap_initializeLocale_tsx failed.
(error: https://s1.sentry-cdn.com/_static/dist/sentry/chunks/app_bootstrap_initializeLocale_tsx.cb92d261ee5cdae6468b.js)

Here's an example.

Expected Result

After removing the ) the URL works:

When detecting URLs maybe we can trim special characters from the end?

Actual Result

The URL which is extracted in the stack trace will include the trailing parenthesis:

[priscilawebdev]

thanks for the heads up @mwarkentin we have added this issue to our backlog

[AbhiPrasad]

We need to adjust the stacktrace parsing regex to fix this! Transferring to the JS SDK as a result.

Tracking JIRA: https://getsentry.atlassian.net/browse/TET-245

[AbhiPrasad]

The raw chrome stacktrace (id's removed to obfiscate):

ChunkLoadError: Loading chunk app_bootstrap_initializeLocale_tsx failed.
(error: https://s1.sentry-cdn.com/_static/dist/sentry/chunks/app_bootstrap_initializeLocale_tsx.abcdefg.js)
  at (error: (/_static/dist/sentry/chunks/app_bootstrap_initializeLocale_tsx.abcdefg.js))
  at key(webpack/runtime/jsonp chunk loading:27:18)
  at ? (webpack/runtime/ensure chunk:6:25)
  at Array.reduce(<anonymous>)
  at Function._.e(webpack/runtime/ensure chunk:5:56)
  at gsApp(../../getsentry/static/getsentry/gsApp/index.tsx:17:7)
  at ? (../../getsentry/static/getsentry/gsApp/index.tsx:49:1)
  at ? (dist/sentry/sourcemaps/app.abcdefg.js:49:1)
  at ? (dist/sentry/sourcemaps/app.abcdefg.js:49:1)

[lobsterkatie]

related:
#5151
#4138
#5498

[timfish]

This stack trace is wild!

Even if I can get the regex to parse this line, the path is relative to https://s1.sentry-cdn.com/ and since ours is a line parser, we have no access to the previous line. This path is then going to look like it's part of the app!
at (error: (/_static/dist/sentry/chunks/app_bootstrap_initializeLocale_tsx.abcdefg.js))

Also, no column or line numbers, so is this even a proper frame?

[AbhiPrasad]

So it seems these are errors being thrown by Webpack.

See:
https://github.com/webpack/webpack/blob/c181294865dca01b28e6e316636fef5f2aad4eb6/test/configCases/web/retry-failed-import/index.js#L15-L20
https://github.com/webpack/webpack/blob/8f87b50dc7ac24eb5c91fd0d55a22e34e252863c/lib/web/JsonpChunkLoadingRuntimeModule.js#L306-L311
https://github.com/webpack/webpack/blob/8f87b50dc7ac24eb5c91fd0d55a22e34e252863c/lib/css/CssLoadingRuntimeModule.js#L316-L322