Skip to content

Add Sentry source map uploads to webpack build#4653

Open
aaccensi wants to merge 1 commit intomasterfrom
sentry-source-maps
Open

Add Sentry source map uploads to webpack build#4653
aaccensi wants to merge 1 commit intomasterfrom
sentry-source-maps

Conversation

@aaccensi
Copy link
Contributor

Description

Configure Sentry source maps so they are passed at build time to Sentry

Type of change

  • Bug fix
  • New feature
  • Enhancement
  • Documentation

Checklist

  • I have tested these changes
  • I have updated documentation as needed
  • I have tagged the relevant people for review

Related Issues

Closes #4652

@aaccensi aaccensi requested a review from noracato February 17, 2026 13:39
@aaccensi aaccensi requested a review from a team as a code owner February 17, 2026 13:39
Copy link
Member

@noracato noracato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One check: like this the token will be included in the Docker image? Or is it just on build?

Because the image pushed to the Docker hub is public. So we should not have our auth token end up in there.

Copy link
Contributor Author

The sentry auth token is only needed at build time, so we do not need it in the docker image. only defined in our deployment method, I already added the following

  • A secret in semaphore for etmodel
  • An env var in Netlify for collections (both beta and pro)

Is worth noting that I used a existing custom internal integration auth token that we had from an earlier test because I believe it will work (based on what this page says) but the docs section of the source maps only mentions organization or personal tokens (I think the internal is simply meant to work like a personal one but for the whole organization), so we should sill make sure the permission are as follows:

  • Release: Admin
  • Project: Read & Write

*Or change it by a organization token, but in any case I do not have enough permission in Sentry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Source maps for Sentry

2 participants