exclude fetchApi "Failed to fetch" from ignores

[oreporan]

Currently, as the API states, you discard instance of TypeError, however, the fetchApi Failed to fetch exception is of type TypeError.
One of the common reasons that this error occurs can be network connection issues.
Is it possible to include this exception?
The main reason I want to add a retry mechanism is incase the user had a bad internet connection during that second where the request was going out

From Fetch documentation:

A fetch() promise will reject with a TypeError when a network error is encountered or CORS is misconfigured on the server-side

Current workaround:

 pRetry(async () => {
        try {
            return await fetchRequest()
        } catch (error) {
            if (error instanceof TypeError && error.message === 'Failed to fetch') {
                throw new Error('Failed to fetch')
            }
            throw error;
        }
    }, {
        retries: 2,
        onFailedAttempt,
    });

Thanks

[sindresorhus]

Who thought it was a good idea for Fetch to throw a TypeError on network errors...

---

Slightly relevant issue: whatwg/fetch#526

[sindresorhus]

But yes, we should exclude this. Also needs to be documented.

[oreporan]

I'm also happy to put a PR in place

[sindresorhus]

Yes. PR welcome :)

[chenliu0831]

+1, this is a bit unexpected when people use retry to mitigate network errors from fetch