Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default backoff of 20 requests leads to long delay in case of error #132

Open
beberlei opened this issue Dec 4, 2024 · 3 comments
Open

Comments

@beberlei
Copy link

beberlei commented Dec 4, 2024

I triggered a 500 error on the ChartMogul backend for some reason, so a call would always regenerate that error, and using the default backoff of 20 this felt like an endless loop, the script was hanging a long time. I believe reducing this to 3 or 5 by default would provide enough value but also end scripts earlier that run into errors on every call.

@swember
Copy link
Contributor

swember commented Dec 4, 2024

Hello @beberlei, thanks a lot for the report. I believe 500 is not a normal situation which shouldn't be happening. Could you please provide which call caused that so we could investigate it further? Related the change of the number of retries we will consider that and prepare an update in the next release.

@beberlei
Copy link
Author

beberlei commented Dec 4, 2024

The problem is a faulty API call on my end, see the "custom" declaration not according to docs.

I sent this to my onboarding rep aswell as an email just now

Bildschirmfoto 2024-12-04 um 15 06 00

The backend error for this is an empty string.

@swember
Copy link
Contributor

swember commented Dec 4, 2024

Thank you @beberlei, we will take a look at the issue with the API and the library.

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

No branches or pull requests

2 participants