Use exponential backoff for s3 503 error

[tuliren]

Resolve #44.

[stoat-app]

Easy and customizable dashboards for your build system. Learn more about Stoat ↗︎

Static Hosting

Name	Link	Commit	Status
Action Test Coverage	Visit	94354cf	✅
Action Test Coverage (single file)	Visit	94354cf	✅

Job Runtime

[tuliren]

Should move this setup to a "beforeEach" block to be consistent across different tests within this file.

[jrhizor]

Looks good.

Seems likely we'll want to use a library for this at some point like https://www.npmjs.com/package/p-retry as we have more operations that require similar retry logic.

[tuliren]

Good point. Will switch to use a library.

[tuliren]

I tried p-retry, but run into some ESM issues, and got this exception:

 FAIL  __tests__/plugins/staticHosting/plugin.test.ts
  ● Test suite failed to run

    Must use import to load ES Module: /Users/lirentu/git/stoat-action/action/node_modules/p-retry/index.js

       6 | // eslint-disable-next-line import/no-unresolved
       7 | import mime from 'mime-types';
    >  8 | import pRetry, { AbortError } from 'p-retry';
         | ^
       9 | import { basename, posix, resolve } from 'path';
      10 |
      11 | import { getApiUrlBase } from '../../stoatApiHelpers';

      at Runtime.requireModule (node_modules/jest-runtime/build/index.js:817:21)
      at Object.<anonymous> (src/plugins/staticHosting/helpers.ts:8:1)
      at Object.<anonymous> (src/plugins/staticHosting/plugin.ts:8:1)
      at Object.<anonymous> (src/plugins/staticHosting/index.ts:1:1)
      at Object.<anonymous> (__tests__/plugins/staticHosting/plugin.test.ts:4:1)

I'd rather not deal with this right now. Created an issue: #47.