dschmidt / ember-cli-deploy-sentry

An ember-cli-deploy-plugin to upload javascript sourcemaps to Sentry
MIT License
42 stars 51 forks source link

Handle pagination when requesting release files from Sentry #78

Closed veelci closed 2 years ago

veelci commented 2 years ago

Updating a release with a more than 100 files fails due to pagination in the Sentry API.

Since only 100 files are deleted before re-uploading, a 409 response is returned when attempting up upload files that already exist.

I think this will fix #73

dschmidt commented 2 years ago

You think or you tried?

dschmidt commented 2 years ago

... and thanks for your PR anyway!

veelci commented 2 years ago

It fixes the 409 conflict I was encountering when trying to update a release with more than 100 files.

I'm not sure if @timjcook's 409 was the same cause.