getsentry / sentry

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

404 from Integration Platform installation API #72380

Open Dhrumil-Sentry opened 5 months ago

Dhrumil-Sentry commented 5 months ago

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

Feedback from Cloudflare:

We've been seeing some occasional issues with certain customers using the Sentry integration. One issue is with POSTing to this endpoint to retrieve a token: https://sentry.io/api/0/sentry-app-installations/{installationId}/authorizations/

We sometimes see it responding with a 404. What are the cases that would cause this to respond that way? One example installationId that I can find in our logs that returns a 404 is e63e4baf-9604-4545-80be-ea953863ef84

Let me know if you need any more info.

Expected Result

.

Actual Result

.

Product Area

Settings - Integrations

Link

No response

DSN

No response

Version

No response

getsantry[bot] commented 4 months ago

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

markjmiller commented 3 months ago

Any timeline for this? We have several customer escalations for the list of Sentry projects not populating in the Workers Sentry Integration and it's because of this API.

Image

leedongwei commented 3 months ago

@markjmiller I've scheduled our bug triage crew to start investigations on this in late Aug.

markjmiller commented 2 months ago

@leedongwei any updates on this?

leedongwei commented 2 months ago

Thanks for the reminder. There's competing priorities for the triage crew, I will get it scheduled again.

wrfengg commented 1 month ago

any progress on this?

GabeVillalobos commented 1 month ago

Hi @markjmiller, @wrfengg. I've started looking into this issue, and the only obvious way I can see it returning a 404 is when we fail to query an active Sentry App installation with the given UUID.

One example installationId that I can find in our logs that returns a 404 is e63e4baf-9604-4545-80be-ea953863ef84

This installation appears to have been soft deleted as of December 2023 which likely explains the 404s you're getting for this particular one.

Any timeline for this? We have several customer escalations for the list of Sentry projects not populating in the Workers Sentry Integration and it's because of this API.

Is there any chance the users affected by this were attempting to reinstall the integration after deleting it?

Any additional context you could provide would be greatly appreciated.

shaunpersad commented 1 month ago

Hi @GabeVillalobos,

Most of the customers experiencing this behavior are installing the integration for the first time, so it would be unlikely that they are trying to reinstall after deleting. I'm working to get a fresh set of logs for you to confirm on your end.

GabeVillalobos commented 1 month ago

@shaunpersad Any updates on the logs? Also, do you have up-to-date documentation on setting up the integration? The previously linked docs are now showing a 404: https://developers.cloudflare.com/workers/observability/sentry-integration/

I have a test worker running and was going to try to triage this issue further.