Retry on 5xx HTTP error

[lojzatran]

nodejs sdk is currently retrying on 503 HTTP error only (

nodejs/packages/sdk-middleware-http/src/http.js

Line 215 in 1167644

if (res.status === 503 && enableRetry)

). According to the CTP documentation it should retry at least on 502 HTTP error (https://docs.commercetools.com/api/errors#502-bad-gateway). You may consider retrying also on other 5xx HTTP errors where it makes sense.

[lojzatran]

Also is it possible to add own retry rules? E.g. we want to retry in some specific 4xx errors that we have confirmed from support that they should be retried. In the old sphere-node-sdk we have retryKeywords and we could also configure them: https://github.com/commercetools/sphere-node-sdk/blob/master/src/coffee/repeater-task-queue.coffee#L7. How could this be done in the current nodejs sdk?

[ajimae]

Hi @lojzatran

I will look into this and if it's something we need to implement I will raise a PR for it.

Thanks

[butenkor]

The question is how reliable is to have error codes only since some of the errors which are listed in the the old keywords SDK are not related to the API response code (since commercetools server is not reached) but also OS limitation like amount of possible concurrent connection and similar - in such cases it would allow the user to retry automatically too.