Retry does not work when using request body

[perjo927]

Hi @estrattonbailey,

I have encountered a problem when using the retry option. It works fine as long as I don't use a request body.

I couldn't figure it out, I debugged and it failed silently in handleRetry. Then I cloned the repo and wrapped a code block in a try/catch:

/* handleRetry.ts */

export async function handleRetry(
	request: () => Promise<Response>,
	method: string,
	retryOptions: Partial<RetryOptions>
) {
	let res;
        // Added try/catch heree
	try {
		res = await request();
	} catch (e) {
		console.log(e);
		return;
	}

Now fetch throws the following error:

Failed to execute 'fetch' on 'Window': Cannot construct a Request with a Request object that has already been used

It appears to happen only when a body is attached to the request, without a request body it works fine.

This answer helped me solve the problem: https://stackoverflow.com/a/55980308/3504096

If I clone the request here: https://github.com/truework/gretchen/blob/master/index.ts#L108-L111

... it works, like so:

/* index.ts */
  const fetcher = () =>
    timeout
      ? handleTimeout(fetch(request.clone()), timeout, controller)
      : fetch(request.clone())

I wanted to submit a pull request but was not authorized to do so.

[perjo927]

Plus, the exponential backoff does not seem to work like described in the docs (6, 36, 216, ...), it generates 6, 36, 1296, 1679616 instead

[perjo927]

Will try forking, for now

[estrattonbailey]

Hey @perjo927 sorry for the delay. Will try to have a look at this this week!

[perjo927]

@estrattonbailey Thanks!

[ssolders]

any update on this? @estrattonbailey