Node retries not compatible with timeouts

I’m looking at integrating superagent to make a server side call to a flaky backend API. The API either responds quickly or doesn’t respond at all and needs to therefore be retried.

I’m therefore using retry(3) in combination with timeout({ deadline: 3000 }). However the timeout is overruling the retry and throwing an Aborted error which returns a 500 error to the client. From what I can tell the following is happening when the request takes too long:

1. Superagent makes a get request to backend API
2. GET request times out so emits an abort
3. abort handler then rejects the request
4. Retry logic kicks in and decides that we should retry
5. Aborted error then causes 500 error to client
6. Backend API is then retried
7. If API call fails then is retried twice more otherwise finishes

Looking at the code the retry logic attempts to reset the _aborted and timedout properties, but it looks like by then it is too late as reject is called.

Is there a way I can therefore retry when the API call times out?

[djmitche]

To reproduce:

const request = require('superagent');

const main = async () => {
  const req = request('http://localhost:8000');
  req.timeout(1000);
  await req.catch(err => {
    if (err.timeout) {
      console.log('got timeout');
    }
    throw err;
  });
};

main().then(console.log, console.log);

and in another terminal, run a server that will not reply. Netcat will do:

nc -klp 8000

[niftylettuce]

v5.3.0 published to npm and GitHub releases page with changelog