fix(sentry): point sourcemap upload at the EU data region #140
No reviewers
Labels
No labels
dependencies
major
security-pin
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
trails-cool/trails!140
Loading…
Reference in a new issue
No description provided.
Delete branch "fix-sentry-eu-region-url"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Fixes the 401 in #5.
The bug
sentry-clisends every API call tohttps://sentry.iounless you tell it otherwise. Our Sentry org is in the EU data region — every DSN in the repo carries ano4509530546634752.ingest.de.sentry.iohost. The US endpoint does not know an EU-issued token, so it answers:That 401 reads like a permissions problem, which is why #5 chased token scopes. But a scope problem gives 403; a 401 means the wrong endpoint. No
SENTRY_URLis set anywhere in the repo and neither vite config passed aurloption, so both apps have always uploaded to the wrong region.The fix
Set the plugin's
urltohttps://de.sentry.ioin bothvite.config.tsfiles. This covers every build path — Docker, CI, local — with no workflow or secrets change. The value sits next to theorgandprojectvalues it belongs with, both of which are already hardcoded.Still open in #5
A failed upload still does not fail the build:
sentry-vite-pluginreports the error and the build continues. That is why this rotted unnoticed for six weeks. The follow-up stays open.Verification
pnpm typecheck,pnpm lintandpnpm testall pass. I did not runpnpm test:e2elocally — it needs the full Docker stack, and this change cannot reach it, because the plugin is disabled whenSENTRY_AUTH_TOKENis absent and the e2e suite never sets one. CI runs it.The real proof is the next production deploy:
cd-appsmust build without the 401, and a release for the new SHA must appear in Sentry. If the 401 continues, the token is bad as well as mis-routed, and it needs a rotation to an Organization Auth Token.Refs #5
🤖 Generated with Claude Code
ullrich referenced this pull request2026-09-11 12:01:08 +00:00