-
Notifications
You must be signed in to change notification settings - Fork 70
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
Fix retry for aborted requests #1713
Comments
@jenschude / @ajimae I was wondering when you are planning to release a new version so we can start using it. |
The regular release cycle is the first monday of a month. So next week monday. |
Oh wait. This repo is auto released once it's merged. So it should be already available |
@jenschude I think that looks like Monday next week. I've checked the current version in NPM and it points to https://www.npmjs.com/package/@commercetools/sdk-middleware-http Unless I'm missing something. |
Hrmpf seems that the autorelease is broken. So we have something to fix for monday 😉 |
@jenschude I was wondering if you managed to fix the autorelease. I still don't see the We would like to start using ASAP if possible. |
New version has been released today. Sorry for the delay, we are updating the release process so it will be more automated again :) https://www.npmjs.com/package/@commercetools/sdk-middleware-http |
Support reference number
N/A
Description
Aborted requests are not being retried when
enableRetry
is enabled because they reuse the same AbortController for try requests.Expected Behavior
Aborted requests are being retried.
Current Behavior
On abort, subsequent aborts are being aborted instantaneously.
Context
When running performance test we see a number of request being aborted which could be due to Commercetools staling-out.
Possible Solution
#1712
The text was updated successfully, but these errors were encountered: