getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
37.71k stars 4.05k forks source link

Can't Map Project in Vercel #72834

Open scefali opened 1 month ago

scefali commented 1 month ago

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

  1. Manually add one of the environment variables used for Sentry (ex: NEXT_PUBLIC_SENTRY_DSN)
  2. Install Vercel
  3. Try to map the project
  4. See error

Note that this bug doesn't happen if Sentry is the one setting the environment variables first (ex: mapping the project twice).

Expected Result

Vercel env variable gets set properly.

Actual Result

Screenshot 2024-06-15 at 9 41 46 AM

It seems like there is a bug somewhere here in updating an existing env variable:

https://github.com/getsentry/sentry/blob/master/src/sentry/integrations/vercel/integration.py#L279-L287

Product Area

Unknown

Link

No response

DSN

No response

Version

No response

getsantry[bot] commented 1 month ago

Assigning to @getsentry/support for routing ⏲️

getsantry[bot] commented 1 month ago

Routing to @getsentry/product-owners-settings-integrations for triage ⏲️

dashed commented 1 month ago

@scefali pull requests are welcomed 😉