Timeout feature

[birdlavv]

Timeout support (limiting the request execution time) would allow for snappy client-facing network timeout error messages if network connectivity drops.

[atinux]

Some context: https://stackoverflow.com/questions/46946380/fetch-api-request-timeout

[atinux]

• https://dmitripavlutin.com/timeout-fetch-request/
• Add a timeout option, to prevent hanging whatwg/fetch#951

[nathanchase]

Would very much like to see timeouts implemented - preferably with an exponential increase in timeout duration on subsequent retries. This is a much needed and essential feature coming from relying on it with axios.

Can we hook into the signal: AbortSignal.timeout(8000) mentioned here: whatwg/fetch#951 (comment)?

Something like:

$fetch(event.req.url, {
    baseURL: config.public.baseUrl,
    params,
    method,
    body,
    retry: 10,
    timeout: 2000, // signal: AbortSignal.timeout(2000),
    ...

[oleynikd]

retryTimeout would be great too

[rmcmk]

+1, would be a very useful feature to have. Currently have to hack around to get this to work in nuxt

[gustavopch]

It could be implemented using https://npm.im/async-retry or adapted directly from the source code of https://github.com/tim-kos/node-retry to avoid adding a dependency.

EDIT: or perhaps just:

-  function onError(context: FetchContext): Promise<FetchResponse<any>> {
+  async function onError(context: FetchContext): Promise<FetchResponse<any>> {
     // Is Abort
     // If it is an active abort, it will not retry automatically.
     // https://developer.mozilla.org/en-US/docs/Web/API/DOMException#error_names
     const isAbort =
       (context.error && context.error.name === "AbortError") || false;
     // Retry
     if (context.options.retry !== false && !isAbort) {
       let retries;
       if (typeof context.options.retry === "number") {
         retries = context.options.retry;
       } else {
         retries = isPayloadMethod(context.options.method) ? 0 : 1;
       }
 
       const responseCode = (context.response && context.response.status) || 500;
       if (retries > 0 && retryStatusCodes.has(responseCode)) {
+        context.options.timeout ??= 1000;
+        await new Promise(resolve => setTimeout(resolve, context.options.timeout));
+        context.options.timeout *= 2;
         return $fetchRaw(context.request, {
           ...context.options,
           retry: retries - 1,
         });
       }
     }

[Siddharth24Khera]

+1, would be a very useful feature to have. Currently have to hack around to get this to work in nuxt

@rmcmk Can you share the workaround you are using in Nuxt.

[danielcmm]

Very much needed!

[seth100]

Looking forward to having this implemented !

[pi0]

implemented in #268 (finally!)