search

getsentry / sentry

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

Modifying uptime monitoring alert fails to save on first click #78185

Open zhiyan114 opened 4 days ago

zhiyan114 commented 4 days ago

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

  1. Create an uptime alert
  2. Refresh inside the Alerts (not sure if it's necessary, but the issue arise after I left and came back to modify it)
  3. Go to the alert that just created and click "Edit Rule"
  4. Modify any parameters and click "Save Rule"

Expected Result

The new modified parameter should be saved on first click.

Actual Result

Observe the network request inside dev tool as the first request are made to https://us.sentry.io/api/0, causing CORS (as provided by the console Access to fetch at 'https://us.sentry.io/api/0' from origin 'https://[OrgSlug].sentry.io' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: Redirect is not allowed for a preflight request.)

Clicking on "Save Rule" again will now properly post the change under https://us.sentry.io/api/0/projects/[OrgSlug]/[ProjectName]/uptime/[EventID]/

Product Area

Alerts

Link

No response

DSN

No response

Version

No response

getsantry[bot] commented 4 days ago

Assigning to @getsentry/support for routing ⏲️

getsantry[bot] commented 4 days ago

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

rachrwang commented 2 days ago

@gaprl - can you help take a look at this & assign to the right person? Thanks!