Can onRetry handle async functions?

[amaury1093]

I have this situation where I download the file, and want to delete this file (async operation) if the downloaded file is corrupted and needs retry.

    return retry(
      async (_, attempt) => {
        await downloadFileFromInternet();
      },
      {
        onRetry: err => {
         // It happens that downloaded file has an error
         // I want to delete the downloaded corrupted file
         await fs.unlink(pathToCorruptedFile);
        },
        retries: 3
      }
    );

Is it possible to have async-retry retry the next attempt only once the onRetry resolves (if the return value is a Promise)?

[BenJackGill]

Any update on this?

I added an async modifier to the onRetry callback but my await functions are not awaited.

To follow on from your example:

    return retry(
      async (_, attempt) => {
        await downloadFileFromInternet();
      },
      {
        onRetry: async (err) => {
         // It happens that downloaded file has an error
         // I want to delete the downloaded corrupted file
         await fs.unlink(pathToCorruptedFile); // This does not wait
        },
        retries: 3
      }
    );

[tvld]

Actually, are you sure? In my case await in onRetry() seems to work just fine

[BenJackGill]

When I was facing this problem I was fairly sure that was the problem. I now use p-retry instead.

[JiriKovar]

I'm sure that the onRetry() is not awaited... It seems we will need to drop the usage of this package because for this reason as there is no workaround and its easier then rewriting everythin we need to use in onRetry to a synchronous implementation.