sentry-javascript icon indicating copy to clipboard operation
sentry-javascript copied to clipboard

Crash-Report Modal 404s with EU DSN

Open Fwang36 opened this issue 10 months ago • 3 comments

Is there an existing issue for this?

  • [X] I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
  • [X] I have reviewed the documentation https://docs.sentry.io/
  • [X] I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/nextjs

SDK Version

7.112.2

Framework Version

next 13

Link to Sentry event

No response

SDK Setup

No response

Steps to Reproduce

I tested in Nextjs, but the user who originally reported it is using Angular-ivy 112.2

  1. Set up Crash Report Modal
  2. Insert EU dsn to init.
  3. call showReportDialog()
  4. GET request 404s

Expected Result

Crash Report Modal to show

Actual Result

Screenshot 2024-04-26 at 9 28 04 AM

Changing the DSN back to a US DSN will work immediately.

Fwang36 avatar Apr 26 '24 13:04 Fwang36

I guess Sentry doesn't route the URL here properly.

https://github.com/getsentry/sentry/blob/fba77f6a0215dcfca40bd0282e3efb52c638c16f/src/sentry/web/frontend/error_page_embed.py#L76-L77

AbhiPrasad avatar Apr 26 '24 13:04 AbhiPrasad

Shouldn't https://github.com/getsentry/sentry-javascript/blob/d381ace59eece2697925b6430af0091e9589503b/packages/core/src/api.ts#L52 be using the host name from the DSN to generate the URL to fetch the error page from? I can patch the server for this, but I would have thought that the DSN would have been enough to get the correct behavior here.

markstory avatar Apr 26 '24 21:04 markstory

Server fix https://github.com/getsentry/sentry/pull/69815

markstory avatar Apr 26 '24 21:04 markstory

The fix for the server endpoints has been deployed, this scenario should be working now for EU organizations.

markstory avatar Apr 30 '24 13:04 markstory