Fix artifact upload chunk timeout logic + update tests#1774
Merged
Conversation
konradpabjan
approved these changes
Jul 24, 2024
jww3
approved these changes
Jul 25, 2024
renovate Bot
added a commit
to sdwilsh/sOS
that referenced
this pull request
Apr 29, 2026
##### [\`v7.0.1\`](https://github.com/actions/upload-artifact/releases/tag/v7.0.1) ##### What's Changed - Update the readme with direct upload details by [@danwkennedy](https://github.com/danwkennedy) in [#795](actions/upload-artifact#795) - Readme: bump all the example versions to v7 by [@danwkennedy](https://github.com/danwkennedy) in [#796](actions/upload-artifact#796) - Include changes in typespec/ts-http-runtime 0.3.5 by [@yacaovsnc](https://github.com/yacaovsnc) in [#797](actions/upload-artifact#797) **Full Changelog**: <actions/upload-artifact@v7...v7.0.1> --- ##### [\`v7.0.0\`](https://github.com/actions/upload-artifact/releases/tag/v7.0.0) #### v7 What's new ##### Direct Uploads Adds support for uploading single files directly (unzipped). Callers can set the new `archive` parameter to `false` to skip zipping the file during upload. Right now, we only support single files. The action will fail if the glob passed resolves to multiple files. The `name` parameter is also ignored with this setting. Instead, the name of the artifact will be the name of the uploaded file. ##### ESM To support new versions of the `@actions/*` packages, we've upgraded the package to ESM. #### What's Changed - Add proxy integration test by [@Link-](https://github.com/Link-) in [#754](actions/upload-artifact#754) - Upgrade the module to ESM and bump dependencies by [@danwkennedy](https://github.com/danwkennedy) in [#762](actions/upload-artifact#762) - Support direct file uploads by [@danwkennedy](https://github.com/danwkennedy) in [#764](actions/upload-artifact#764) #### New Contributors - [@Link-](https://github.com/Link-) made their first contribution in [#754](actions/upload-artifact#754) **Full Changelog**: <actions/upload-artifact@v6...v7.0.0> --- ##### [\`v7\`](actions/upload-artifact@v6...v7) --- ##### [\`v6\`](actions/upload-artifact@v5...v6) --- ##### [\`v6.0.0\`](actions/upload-artifact@v5.0.0...v6.0.0) --- ##### [\`v5\`](actions/upload-artifact@v4...v5) --- ##### [\`v5.0.0\`](actions/upload-artifact@v4.6.2...v5.0.0) --- ##### [\`v4.6.2\`](https://github.com/actions/upload-artifact/releases/tag/v4.6.2) #### What's Changed - Update to use artifact 2.3.2 package & prepare for new upload-artifact release by [@salmanmkc](https://github.com/salmanmkc) in [#685](actions/upload-artifact#685) #### New Contributors - [@salmanmkc](https://github.com/salmanmkc) made their first contribution in [#685](actions/upload-artifact#685) **Full Changelog**: <actions/upload-artifact@v4...v4.6.2> --- ##### [\`v4.6.1\`](https://github.com/actions/upload-artifact/releases/tag/v4.6.1) #### What's Changed - Update to use artifact 2.2.2 package by [@yacaovsnc](https://github.com/yacaovsnc) in [#673](actions/upload-artifact#673) **Full Changelog**: <actions/upload-artifact@v4...v4.6.1> --- ##### [\`v4.6.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.6.0) #### What's Changed - Expose env vars to control concurrency and timeout by [@yacaovsnc](https://github.com/yacaovsnc) in [#662](actions/upload-artifact#662) **Full Changelog**: <actions/upload-artifact@v4...v4.6.0> --- ##### [\`v4.5.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.5.0) #### What's Changed - fix: deprecated `Node.js` version in action by [@hamirmahal](https://github.com/hamirmahal) in [#578](actions/upload-artifact#578) - Add new `artifact-digest` output by [@bdehamer](https://github.com/bdehamer) in [#656](actions/upload-artifact#656) #### New Contributors - [@hamirmahal](https://github.com/hamirmahal) made their first contribution in [#578](actions/upload-artifact#578) - [@bdehamer](https://github.com/bdehamer) made their first contribution in [#656](actions/upload-artifact#656) **Full Changelog**: <actions/upload-artifact@v4.4.3...v4.5.0> --- ##### [\`v4.4.3\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.3) #### What's Changed - Undo indirect dependency updates from [#627](actions/upload-artifact#627) by [@joshmgross](https://github.com/joshmgross) in [#632](actions/upload-artifact#632) **Full Changelog**: <actions/upload-artifact@v4.4.2...v4.4.3> --- ##### [\`v4.4.2\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.2) #### What's Changed - Bump `@actions/artifact` to 2.1.11 by [@robherley](https://github.com/robherley) in [#627](actions/upload-artifact#627) - Includes fix for relative symlinks not resolving properly **Full Changelog**: <actions/upload-artifact@v4.4.1...v4.4.2> --- ##### [\`v4.4.1\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.1) #### What's Changed - Add a section about hidden files by [@joshmgross](https://github.com/joshmgross) in [#607](actions/upload-artifact#607) - Add workflow file for publishing releases to immutable action package by [@Jcambass](https://github.com/Jcambass) in [#621](actions/upload-artifact#621) - Update [@actions/artifact](https://github.com/actions/artifact) to latest version, includes symlink and timeout fixes by [@robherley](https://github.com/robherley) in [#625](actions/upload-artifact#625) #### New Contributors - [@Jcambass](https://github.com/Jcambass) made their first contribution in [#621](actions/upload-artifact#621) **Full Changelog**: <actions/upload-artifact@v4.4.0...v4.4.1> --- ##### [\`v4.4.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.0) #### Notice: Breaking Changes⚠️ We will no longer include hidden files and folders by default in the `upload-artifact` action of this version. This reduces the risk that credentials are accidentally uploaded into artifacts. Customers who need to continue to upload these files can use a new option, `include-hidden-files`, to continue to do so. See ["Notice of upcoming deprecations and breaking changes in GitHub Actions runners"](https://github.blog/changelog/2024-08-19-notice-of-upcoming-deprecations-and-breaking-changes-in-github-actions-runners/) changelog and [this issue](actions/upload-artifact#602) for more details. #### What's Changed - Exclude hidden files by default by [@joshmgross](https://github.com/joshmgross) in [#598](actions/upload-artifact#598) **Full Changelog**: <actions/upload-artifact@v4.3.6...v4.4.0> --- ##### [\`v4.3.6\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.6) #### What's Changed - Revert to [@actions/artifact](https://github.com/actions/artifact) 2.1.8 by [@robherley](https://github.com/robherley) in [#594](actions/upload-artifact#594) **Full Changelog**: <actions/upload-artifact@v4...v4.3.6> --- ##### [\`v4.3.5\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.5) #### What's Changed - Bump [@actions/artifact](https://github.com/actions/artifact) to v2.1.9 by [@robherley](https://github.com/robherley) in [#588](actions/upload-artifact#588) - Fixed artifact upload chunk timeout logic [#1774](actions/toolkit#1774) - Use lazy stream to prevent issues with open file limits [#1771](actions/toolkit#1771) **Full Changelog**: <actions/upload-artifact@v4.3.4...v4.3.5> --- ##### [\`v4.3.4\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.4) #### What's Changed - Update [@actions/artifact](https://github.com/actions/artifact) version, bump dependencies by [@robherley](https://github.com/robherley) in [#584](actions/upload-artifact#584) **Full Changelog**: <actions/upload-artifact@v4.3.3...v4.3.4> --- ##### [\`v4.3.3\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.3) #### What's Changed - updating `@actions/artifact` dependency to v2.1.6 by [@eggyhead](https://github.com/eggyhead) in [#565](actions/upload-artifact#565) **Full Changelog**: <actions/upload-artifact@v4.3.2...v4.3.3> --- ##### [\`v4.3.2\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.2) #### What's Changed - Update release-new-action-version.yml by [@konradpabjan](https://github.com/konradpabjan) in [#516](actions/upload-artifact#516) - Minor fix to the migration readme by [@andrewakim](https://github.com/andrewakim) in [#523](actions/upload-artifact#523) - Update readme with v3/v2/v1 deprecation notice by [@robherley](https://github.com/robherley) in [#561](actions/upload-artifact#561) - updating `@actions/artifact` dependency to v2.1.5 and `@actions/core` to v1.0.1 by [@eggyhead](https://github.com/eggyhead) in [#562](actions/upload-artifact#562) #### New Contributors - [@andrewakim](https://github.com/andrewakim) made their first contribution in [#523](actions/upload-artifact#523) **Full Changelog**: <actions/upload-artifact@v4.3.1...v4.3.2> --- ##### [\`v4.3.1\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.1) - Bump [@actions/artifacts](https://github.com/actions/artifacts) to latest version to include [updated GHES host check](actions/toolkit#1648) --- ##### [\`v4.3.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.0) #### What's Changed - Reorganize upload code in prep for merge logic & add more tests by [@robherley](https://github.com/robherley) in [#504](actions/upload-artifact#504) - Add sub-action to merge artifacts by [@robherley](https://github.com/robherley) in [#505](actions/upload-artifact#505) **Full Changelog**: <actions/upload-artifact@v4...v4.3.0> --- ##### [\`v4.2.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.2.0) #### What's Changed - Ability to overwrite an Artifact by [@robherley](https://github.com/robherley) in [#501](actions/upload-artifact#501) **Full Changelog**: <actions/upload-artifact@v4...v4.2.0> --- ##### [\`v4.1.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.1.0) #### What's Changed - Add migrations docs by [@robherley](https://github.com/robherley) in [#482](actions/upload-artifact#482) - Update README.md by [@samuelwine](https://github.com/samuelwine) in [#492](actions/upload-artifact#492) - Support artifact-url output by [@konradpabjan](https://github.com/konradpabjan) in [#496](actions/upload-artifact#496) - Update readme to reflect new 500 artifact per job limit by [@robherley](https://github.com/robherley) in [#497](actions/upload-artifact#497) #### New Contributors - [@samuelwine](https://github.com/samuelwine) made their first contribution in [#492](actions/upload-artifact#492) **Full Changelog**: <actions/upload-artifact@v4...v4.1.0>
renovate Bot
added a commit
to sdwilsh/sOS
that referenced
this pull request
May 12, 2026
##### [\`v7.0.1\`](https://github.com/actions/upload-artifact/releases/tag/v7.0.1) ##### What's Changed - Update the readme with direct upload details by [@danwkennedy](https://github.com/danwkennedy) in [#795](actions/upload-artifact#795) - Readme: bump all the example versions to v7 by [@danwkennedy](https://github.com/danwkennedy) in [#796](actions/upload-artifact#796) - Include changes in typespec/ts-http-runtime 0.3.5 by [@yacaovsnc](https://github.com/yacaovsnc) in [#797](actions/upload-artifact#797) **Full Changelog**: <actions/upload-artifact@v7...v7.0.1> --- ##### [\`v7.0.0\`](https://github.com/actions/upload-artifact/releases/tag/v7.0.0) #### v7 What's new ##### Direct Uploads Adds support for uploading single files directly (unzipped). Callers can set the new `archive` parameter to `false` to skip zipping the file during upload. Right now, we only support single files. The action will fail if the glob passed resolves to multiple files. The `name` parameter is also ignored with this setting. Instead, the name of the artifact will be the name of the uploaded file. ##### ESM To support new versions of the `@actions/*` packages, we've upgraded the package to ESM. #### What's Changed - Add proxy integration test by [@Link-](https://github.com/Link-) in [#754](actions/upload-artifact#754) - Upgrade the module to ESM and bump dependencies by [@danwkennedy](https://github.com/danwkennedy) in [#762](actions/upload-artifact#762) - Support direct file uploads by [@danwkennedy](https://github.com/danwkennedy) in [#764](actions/upload-artifact#764) #### New Contributors - [@Link-](https://github.com/Link-) made their first contribution in [#754](actions/upload-artifact#754) **Full Changelog**: <actions/upload-artifact@v6...v7.0.0> --- ##### [\`v7\`](actions/upload-artifact@v6...v7) --- ##### [\`v6.0.0\`](actions/upload-artifact@v5.0.0...v6.0.0) --- ##### [\`v6\`](actions/upload-artifact@v5...v6) --- ##### [\`v5.0.0\`](actions/upload-artifact@v4.6.2...v5.0.0) --- ##### [\`v5\`](actions/upload-artifact@v4...v5) --- ##### [\`v4.6.2\`](https://github.com/actions/upload-artifact/releases/tag/v4.6.2) #### What's Changed - Update to use artifact 2.3.2 package & prepare for new upload-artifact release by [@salmanmkc](https://github.com/salmanmkc) in [#685](actions/upload-artifact#685) #### New Contributors - [@salmanmkc](https://github.com/salmanmkc) made their first contribution in [#685](actions/upload-artifact#685) **Full Changelog**: <actions/upload-artifact@v4...v4.6.2> --- ##### [\`v4.6.1\`](https://github.com/actions/upload-artifact/releases/tag/v4.6.1) #### What's Changed - Update to use artifact 2.2.2 package by [@yacaovsnc](https://github.com/yacaovsnc) in [#673](actions/upload-artifact#673) **Full Changelog**: <actions/upload-artifact@v4...v4.6.1> --- ##### [\`v4.6.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.6.0) #### What's Changed - Expose env vars to control concurrency and timeout by [@yacaovsnc](https://github.com/yacaovsnc) in [#662](actions/upload-artifact#662) **Full Changelog**: <actions/upload-artifact@v4...v4.6.0> --- ##### [\`v4.5.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.5.0) #### What's Changed - fix: deprecated `Node.js` version in action by [@hamirmahal](https://github.com/hamirmahal) in [#578](actions/upload-artifact#578) - Add new `artifact-digest` output by [@bdehamer](https://github.com/bdehamer) in [#656](actions/upload-artifact#656) #### New Contributors - [@hamirmahal](https://github.com/hamirmahal) made their first contribution in [#578](actions/upload-artifact#578) - [@bdehamer](https://github.com/bdehamer) made their first contribution in [#656](actions/upload-artifact#656) **Full Changelog**: <actions/upload-artifact@v4.4.3...v4.5.0> --- ##### [\`v4.4.3\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.3) #### What's Changed - Undo indirect dependency updates from [#627](actions/upload-artifact#627) by [@joshmgross](https://github.com/joshmgross) in [#632](actions/upload-artifact#632) **Full Changelog**: <actions/upload-artifact@v4.4.2...v4.4.3> --- ##### [\`v4.4.2\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.2) #### What's Changed - Bump `@actions/artifact` to 2.1.11 by [@robherley](https://github.com/robherley) in [#627](actions/upload-artifact#627) - Includes fix for relative symlinks not resolving properly **Full Changelog**: <actions/upload-artifact@v4.4.1...v4.4.2> --- ##### [\`v4.4.1\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.1) #### What's Changed - Add a section about hidden files by [@joshmgross](https://github.com/joshmgross) in [#607](actions/upload-artifact#607) - Add workflow file for publishing releases to immutable action package by [@Jcambass](https://github.com/Jcambass) in [#621](actions/upload-artifact#621) - Update [@actions/artifact](https://github.com/actions/artifact) to latest version, includes symlink and timeout fixes by [@robherley](https://github.com/robherley) in [#625](actions/upload-artifact#625) #### New Contributors - [@Jcambass](https://github.com/Jcambass) made their first contribution in [#621](actions/upload-artifact#621) **Full Changelog**: <actions/upload-artifact@v4.4.0...v4.4.1> --- ##### [\`v4.4.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.0) #### Notice: Breaking Changes⚠️ We will no longer include hidden files and folders by default in the `upload-artifact` action of this version. This reduces the risk that credentials are accidentally uploaded into artifacts. Customers who need to continue to upload these files can use a new option, `include-hidden-files`, to continue to do so. See ["Notice of upcoming deprecations and breaking changes in GitHub Actions runners"](https://github.blog/changelog/2024-08-19-notice-of-upcoming-deprecations-and-breaking-changes-in-github-actions-runners/) changelog and [this issue](actions/upload-artifact#602) for more details. #### What's Changed - Exclude hidden files by default by [@joshmgross](https://github.com/joshmgross) in [#598](actions/upload-artifact#598) **Full Changelog**: <actions/upload-artifact@v4.3.6...v4.4.0> --- ##### [\`v4.3.6\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.6) #### What's Changed - Revert to [@actions/artifact](https://github.com/actions/artifact) 2.1.8 by [@robherley](https://github.com/robherley) in [#594](actions/upload-artifact#594) **Full Changelog**: <actions/upload-artifact@v4...v4.3.6> --- ##### [\`v4.3.5\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.5) #### What's Changed - Bump [@actions/artifact](https://github.com/actions/artifact) to v2.1.9 by [@robherley](https://github.com/robherley) in [#588](actions/upload-artifact#588) - Fixed artifact upload chunk timeout logic [#1774](actions/toolkit#1774) - Use lazy stream to prevent issues with open file limits [#1771](actions/toolkit#1771) **Full Changelog**: <actions/upload-artifact@v4.3.4...v4.3.5> --- ##### [\`v4.3.4\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.4) #### What's Changed - Update [@actions/artifact](https://github.com/actions/artifact) version, bump dependencies by [@robherley](https://github.com/robherley) in [#584](actions/upload-artifact#584) **Full Changelog**: <actions/upload-artifact@v4.3.3...v4.3.4> --- ##### [\`v4.3.3\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.3) #### What's Changed - updating `@actions/artifact` dependency to v2.1.6 by [@eggyhead](https://github.com/eggyhead) in [#565](actions/upload-artifact#565) **Full Changelog**: <actions/upload-artifact@v4.3.2...v4.3.3> --- ##### [\`v4.3.2\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.2) #### What's Changed - Update release-new-action-version.yml by [@konradpabjan](https://github.com/konradpabjan) in [#516](actions/upload-artifact#516) - Minor fix to the migration readme by [@andrewakim](https://github.com/andrewakim) in [#523](actions/upload-artifact#523) - Update readme with v3/v2/v1 deprecation notice by [@robherley](https://github.com/robherley) in [#561](actions/upload-artifact#561) - updating `@actions/artifact` dependency to v2.1.5 and `@actions/core` to v1.0.1 by [@eggyhead](https://github.com/eggyhead) in [#562](actions/upload-artifact#562) #### New Contributors - [@andrewakim](https://github.com/andrewakim) made their first contribution in [#523](actions/upload-artifact#523) **Full Changelog**: <actions/upload-artifact@v4.3.1...v4.3.2> --- ##### [\`v4.3.1\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.1) - Bump [@actions/artifacts](https://github.com/actions/artifacts) to latest version to include [updated GHES host check](actions/toolkit#1648) --- ##### [\`v4.3.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.0) #### What's Changed - Reorganize upload code in prep for merge logic & add more tests by [@robherley](https://github.com/robherley) in [#504](actions/upload-artifact#504) - Add sub-action to merge artifacts by [@robherley](https://github.com/robherley) in [#505](actions/upload-artifact#505) **Full Changelog**: <actions/upload-artifact@v4...v4.3.0> --- ##### [\`v4.2.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.2.0) #### What's Changed - Ability to overwrite an Artifact by [@robherley](https://github.com/robherley) in [#501](actions/upload-artifact#501) **Full Changelog**: <actions/upload-artifact@v4...v4.2.0> --- ##### [\`v4.1.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.1.0) #### What's Changed - Add migrations docs by [@robherley](https://github.com/robherley) in [#482](actions/upload-artifact#482) - Update README.md by [@samuelwine](https://github.com/samuelwine) in [#492](actions/upload-artifact#492) - Support artifact-url output by [@konradpabjan](https://github.com/konradpabjan) in [#496](actions/upload-artifact#496) - Update readme to reflect new 500 artifact per job limit by [@robherley](https://github.com/robherley) in [#497](actions/upload-artifact#497) #### New Contributors - [@samuelwine](https://github.com/samuelwine) made their first contribution in [#492](actions/upload-artifact#492) **Full Changelog**: <actions/upload-artifact@v4...v4.1.0>
renovate Bot
added a commit
to sdwilsh/sOS
that referenced
this pull request
May 22, 2026
##### [\`v7.0.1\`](https://github.com/actions/upload-artifact/releases/tag/v7.0.1) ##### What's Changed - Update the readme with direct upload details by [@danwkennedy](https://github.com/danwkennedy) in [#795](actions/upload-artifact#795) - Readme: bump all the example versions to v7 by [@danwkennedy](https://github.com/danwkennedy) in [#796](actions/upload-artifact#796) - Include changes in typespec/ts-http-runtime 0.3.5 by [@yacaovsnc](https://github.com/yacaovsnc) in [#797](actions/upload-artifact#797) **Full Changelog**: <actions/upload-artifact@v7...v7.0.1> --- ##### [\`v7.0.0\`](https://github.com/actions/upload-artifact/releases/tag/v7.0.0) #### v7 What's new ##### Direct Uploads Adds support for uploading single files directly (unzipped). Callers can set the new `archive` parameter to `false` to skip zipping the file during upload. Right now, we only support single files. The action will fail if the glob passed resolves to multiple files. The `name` parameter is also ignored with this setting. Instead, the name of the artifact will be the name of the uploaded file. ##### ESM To support new versions of the `@actions/*` packages, we've upgraded the package to ESM. #### What's Changed - Add proxy integration test by [@Link-](https://github.com/Link-) in [#754](actions/upload-artifact#754) - Upgrade the module to ESM and bump dependencies by [@danwkennedy](https://github.com/danwkennedy) in [#762](actions/upload-artifact#762) - Support direct file uploads by [@danwkennedy](https://github.com/danwkennedy) in [#764](actions/upload-artifact#764) #### New Contributors - [@Link-](https://github.com/Link-) made their first contribution in [#754](actions/upload-artifact#754) **Full Changelog**: <actions/upload-artifact@v6...v7.0.0> --- ##### [\`v7\`](actions/upload-artifact@v6...v7) --- ##### [\`v6.0.0\`](actions/upload-artifact@v5.0.0...v6.0.0) --- ##### [\`v6\`](actions/upload-artifact@v5...v6) --- ##### [\`v5.0.0\`](actions/upload-artifact@v4.6.2...v5.0.0) --- ##### [\`v5\`](actions/upload-artifact@v4...v5) --- ##### [\`v4.6.2\`](https://github.com/actions/upload-artifact/releases/tag/v4.6.2) #### What's Changed - Update to use artifact 2.3.2 package & prepare for new upload-artifact release by [@salmanmkc](https://github.com/salmanmkc) in [#685](actions/upload-artifact#685) #### New Contributors - [@salmanmkc](https://github.com/salmanmkc) made their first contribution in [#685](actions/upload-artifact#685) **Full Changelog**: <actions/upload-artifact@v4...v4.6.2> --- ##### [\`v4.6.1\`](https://github.com/actions/upload-artifact/releases/tag/v4.6.1) #### What's Changed - Update to use artifact 2.2.2 package by [@yacaovsnc](https://github.com/yacaovsnc) in [#673](actions/upload-artifact#673) **Full Changelog**: <actions/upload-artifact@v4...v4.6.1> --- ##### [\`v4.6.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.6.0) #### What's Changed - Expose env vars to control concurrency and timeout by [@yacaovsnc](https://github.com/yacaovsnc) in [#662](actions/upload-artifact#662) **Full Changelog**: <actions/upload-artifact@v4...v4.6.0> --- ##### [\`v4.5.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.5.0) #### What's Changed - fix: deprecated `Node.js` version in action by [@hamirmahal](https://github.com/hamirmahal) in [#578](actions/upload-artifact#578) - Add new `artifact-digest` output by [@bdehamer](https://github.com/bdehamer) in [#656](actions/upload-artifact#656) #### New Contributors - [@hamirmahal](https://github.com/hamirmahal) made their first contribution in [#578](actions/upload-artifact#578) - [@bdehamer](https://github.com/bdehamer) made their first contribution in [#656](actions/upload-artifact#656) **Full Changelog**: <actions/upload-artifact@v4.4.3...v4.5.0> --- ##### [\`v4.4.3\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.3) #### What's Changed - Undo indirect dependency updates from [#627](actions/upload-artifact#627) by [@joshmgross](https://github.com/joshmgross) in [#632](actions/upload-artifact#632) **Full Changelog**: <actions/upload-artifact@v4.4.2...v4.4.3> --- ##### [\`v4.4.2\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.2) #### What's Changed - Bump `@actions/artifact` to 2.1.11 by [@robherley](https://github.com/robherley) in [#627](actions/upload-artifact#627) - Includes fix for relative symlinks not resolving properly **Full Changelog**: <actions/upload-artifact@v4.4.1...v4.4.2> --- ##### [\`v4.4.1\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.1) #### What's Changed - Add a section about hidden files by [@joshmgross](https://github.com/joshmgross) in [#607](actions/upload-artifact#607) - Add workflow file for publishing releases to immutable action package by [@Jcambass](https://github.com/Jcambass) in [#621](actions/upload-artifact#621) - Update [@actions/artifact](https://github.com/actions/artifact) to latest version, includes symlink and timeout fixes by [@robherley](https://github.com/robherley) in [#625](actions/upload-artifact#625) #### New Contributors - [@Jcambass](https://github.com/Jcambass) made their first contribution in [#621](actions/upload-artifact#621) **Full Changelog**: <actions/upload-artifact@v4.4.0...v4.4.1> --- ##### [\`v4.4.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.4.0) #### Notice: Breaking Changes⚠️ We will no longer include hidden files and folders by default in the `upload-artifact` action of this version. This reduces the risk that credentials are accidentally uploaded into artifacts. Customers who need to continue to upload these files can use a new option, `include-hidden-files`, to continue to do so. See ["Notice of upcoming deprecations and breaking changes in GitHub Actions runners"](https://github.blog/changelog/2024-08-19-notice-of-upcoming-deprecations-and-breaking-changes-in-github-actions-runners/) changelog and [this issue](actions/upload-artifact#602) for more details. #### What's Changed - Exclude hidden files by default by [@joshmgross](https://github.com/joshmgross) in [#598](actions/upload-artifact#598) **Full Changelog**: <actions/upload-artifact@v4.3.6...v4.4.0> --- ##### [\`v4.3.6\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.6) #### What's Changed - Revert to [@actions/artifact](https://github.com/actions/artifact) 2.1.8 by [@robherley](https://github.com/robherley) in [#594](actions/upload-artifact#594) **Full Changelog**: <actions/upload-artifact@v4...v4.3.6> --- ##### [\`v4.3.5\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.5) #### What's Changed - Bump [@actions/artifact](https://github.com/actions/artifact) to v2.1.9 by [@robherley](https://github.com/robherley) in [#588](actions/upload-artifact#588) - Fixed artifact upload chunk timeout logic [#1774](actions/toolkit#1774) - Use lazy stream to prevent issues with open file limits [#1771](actions/toolkit#1771) **Full Changelog**: <actions/upload-artifact@v4.3.4...v4.3.5> --- ##### [\`v4.3.4\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.4) #### What's Changed - Update [@actions/artifact](https://github.com/actions/artifact) version, bump dependencies by [@robherley](https://github.com/robherley) in [#584](actions/upload-artifact#584) **Full Changelog**: <actions/upload-artifact@v4.3.3...v4.3.4> --- ##### [\`v4.3.3\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.3) #### What's Changed - updating `@actions/artifact` dependency to v2.1.6 by [@eggyhead](https://github.com/eggyhead) in [#565](actions/upload-artifact#565) **Full Changelog**: <actions/upload-artifact@v4.3.2...v4.3.3> --- ##### [\`v4.3.2\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.2) #### What's Changed - Update release-new-action-version.yml by [@konradpabjan](https://github.com/konradpabjan) in [#516](actions/upload-artifact#516) - Minor fix to the migration readme by [@andrewakim](https://github.com/andrewakim) in [#523](actions/upload-artifact#523) - Update readme with v3/v2/v1 deprecation notice by [@robherley](https://github.com/robherley) in [#561](actions/upload-artifact#561) - updating `@actions/artifact` dependency to v2.1.5 and `@actions/core` to v1.0.1 by [@eggyhead](https://github.com/eggyhead) in [#562](actions/upload-artifact#562) #### New Contributors - [@andrewakim](https://github.com/andrewakim) made their first contribution in [#523](actions/upload-artifact#523) **Full Changelog**: <actions/upload-artifact@v4.3.1...v4.3.2> --- ##### [\`v4.3.1\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.1) - Bump [@actions/artifacts](https://github.com/actions/artifacts) to latest version to include [updated GHES host check](actions/toolkit#1648) --- ##### [\`v4.3.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.3.0) #### What's Changed - Reorganize upload code in prep for merge logic & add more tests by [@robherley](https://github.com/robherley) in [#504](actions/upload-artifact#504) - Add sub-action to merge artifacts by [@robherley](https://github.com/robherley) in [#505](actions/upload-artifact#505) **Full Changelog**: <actions/upload-artifact@v4...v4.3.0> --- ##### [\`v4.2.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.2.0) #### What's Changed - Ability to overwrite an Artifact by [@robherley](https://github.com/robherley) in [#501](actions/upload-artifact#501) **Full Changelog**: <actions/upload-artifact@v4...v4.2.0> --- ##### [\`v4.1.0\`](https://github.com/actions/upload-artifact/releases/tag/v4.1.0) #### What's Changed - Add migrations docs by [@robherley](https://github.com/robherley) in [#482](actions/upload-artifact#482) - Update README.md by [@samuelwine](https://github.com/samuelwine) in [#492](actions/upload-artifact#492) - Support artifact-url output by [@konradpabjan](https://github.com/konradpabjan) in [#496](actions/upload-artifact#496) - Update readme to reflect new 500 artifact per job limit by [@robherley](https://github.com/robherley) in [#497](actions/upload-artifact#497) #### New Contributors - [@samuelwine](https://github.com/samuelwine) made their first contribution in [#492](actions/upload-artifact#492) **Full Changelog**: <actions/upload-artifact@v4...v4.1.0>
luketainton
pushed a commit
to luketainton/repos_epage-go
that referenced
this pull request
Jun 19, 2026
This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [actions/upload-artifact](https://github.com/actions/upload-artifact) | action | major | `v3` → `v7` | --- ### Release Notes <details> <summary>actions/upload-artifact (actions/upload-artifact)</summary> ### [`v7.0.1`](https://github.com/actions/upload-artifact/releases/tag/v7.0.1) [Compare Source](actions/upload-artifact@v7...v7.0.1) #### What's Changed - Update the readme with direct upload details by [@​danwkennedy](https://github.com/danwkennedy) in [#​795](actions/upload-artifact#795) - Readme: bump all the example versions to v7 by [@​danwkennedy](https://github.com/danwkennedy) in [#​796](actions/upload-artifact#796) - Include changes in typespec/ts-http-runtime 0.3.5 by [@​yacaovsnc](https://github.com/yacaovsnc) in [#​797](actions/upload-artifact#797) **Full Changelog**: <actions/upload-artifact@v7...v7.0.1> ### [`v7.0.0`](https://github.com/actions/upload-artifact/releases/tag/v7.0.0) [Compare Source](actions/upload-artifact@v7...v7) #### v7 What's new ##### Direct Uploads Adds support for uploading single files directly (unzipped). Callers can set the new `archive` parameter to `false` to skip zipping the file during upload. Right now, we only support single files. The action will fail if the glob passed resolves to multiple files. The `name` parameter is also ignored with this setting. Instead, the name of the artifact will be the name of the uploaded file. ##### ESM To support new versions of the `@actions/*` packages, we've upgraded the package to ESM. #### What's Changed - Add proxy integration test by [@​Link-](https://github.com/Link-) in [#​754](actions/upload-artifact#754) - Upgrade the module to ESM and bump dependencies by [@​danwkennedy](https://github.com/danwkennedy) in [#​762](actions/upload-artifact#762) - Support direct file uploads by [@​danwkennedy](https://github.com/danwkennedy) in [#​764](actions/upload-artifact#764) #### New Contributors - [@​Link-](https://github.com/Link-) made their first contribution in [#​754](actions/upload-artifact#754) **Full Changelog**: <actions/upload-artifact@v6...v7.0.0> ### [`v7`](actions/upload-artifact@v6.0.0...v7) [Compare Source](actions/upload-artifact@v6.0.0...v7) ### [`v6.0.0`](https://github.com/actions/upload-artifact/releases/tag/v6.0.0) [Compare Source](actions/upload-artifact@v6.0.0...v6.0.0) #### v6 - What's new > \[!IMPORTANT] > actions/upload-artifact\@​v6 now runs on Node.js 24 (`runs.using: node24`) and requires a minimum Actions Runner version of 2.327.1. If you are using self-hosted runners, ensure they are updated before upgrading. ##### Node.js 24 This release updates the runtime to Node.js 24. v5 had preliminary support for Node.js 24, however this action was by default still running on Node.js 20. Now this action by default will run on Node.js 24. #### What's Changed - Upload Artifact Node 24 support by [@​salmanmkc](https://github.com/salmanmkc) in [#​719](actions/upload-artifact#719) - fix: update [@​actions/artifact](https://github.com/actions/artifact) for Node.js 24 punycode deprecation by [@​salmanmkc](https://github.com/salmanmkc) in [#​744](actions/upload-artifact#744) - prepare release v6.0.0 for Node.js 24 support by [@​salmanmkc](https://github.com/salmanmkc) in [#​745](actions/upload-artifact#745) **Full Changelog**: <actions/upload-artifact@v5.0.0...v6.0.0> ### [`v6`](actions/upload-artifact@v5.0.0...v6.0.0) [Compare Source](actions/upload-artifact@v5.0.0...v6.0.0) ### [`v5.0.0`](https://github.com/actions/upload-artifact/releases/tag/v5.0.0) [Compare Source](actions/upload-artifact@v5.0.0...v5.0.0) #### What's Changed **BREAKING CHANGE:** this update supports Node `v24.x`. This is not a breaking change per-se but we're treating it as such. - Update README.md by [@​GhadimiR](https://github.com/GhadimiR) in [#​681](actions/upload-artifact#681) - Update README.md by [@​nebuk89](https://github.com/nebuk89) in [#​712](actions/upload-artifact#712) - Readme: spell out the first use of GHES by [@​danwkennedy](https://github.com/danwkennedy) in [#​727](actions/upload-artifact#727) - Update GHES guidance to include reference to Node 20 version by [@​patrikpolyak](https://github.com/patrikpolyak) in [#​725](actions/upload-artifact#725) - Bump `@actions/artifact` to `v4.0.0` - Prepare `v5.0.0` by [@​danwkennedy](https://github.com/danwkennedy) in [#​734](actions/upload-artifact#734) #### New Contributors - [@​GhadimiR](https://github.com/GhadimiR) made their first contribution in [#​681](actions/upload-artifact#681) - [@​nebuk89](https://github.com/nebuk89) made their first contribution in [#​712](actions/upload-artifact#712) - [@​danwkennedy](https://github.com/danwkennedy) made their first contribution in [#​727](actions/upload-artifact#727) - [@​patrikpolyak](https://github.com/patrikpolyak) made their first contribution in [#​725](actions/upload-artifact#725) **Full Changelog**: <actions/upload-artifact@v4...v5.0.0> ### [`v5`](actions/upload-artifact@v4.6.2...v5.0.0) [Compare Source](actions/upload-artifact@v4.6.2...v5.0.0) ### [`v4.6.2`](https://github.com/actions/upload-artifact/releases/tag/v4.6.2) [Compare Source](actions/upload-artifact@v4.6.1...v4.6.2) #### What's Changed - Update to use artifact 2.3.2 package & prepare for new upload-artifact release by [@​salmanmkc](https://github.com/salmanmkc) in [#​685](actions/upload-artifact#685) #### New Contributors - [@​salmanmkc](https://github.com/salmanmkc) made their first contribution in [#​685](actions/upload-artifact#685) **Full Changelog**: <actions/upload-artifact@v4...v4.6.2> ### [`v4.6.1`](https://github.com/actions/upload-artifact/releases/tag/v4.6.1) [Compare Source](actions/upload-artifact@v4.6.0...v4.6.1) #### What's Changed - Update to use artifact 2.2.2 package by [@​yacaovsnc](https://github.com/yacaovsnc) in [#​673](actions/upload-artifact#673) **Full Changelog**: <actions/upload-artifact@v4...v4.6.1> ### [`v4.6.0`](https://github.com/actions/upload-artifact/releases/tag/v4.6.0) [Compare Source](actions/upload-artifact@v4.5.0...v4.6.0) #### What's Changed - Expose env vars to control concurrency and timeout by [@​yacaovsnc](https://github.com/yacaovsnc) in [#​662](actions/upload-artifact#662) **Full Changelog**: <actions/upload-artifact@v4...v4.6.0> ### [`v4.5.0`](https://github.com/actions/upload-artifact/releases/tag/v4.5.0) [Compare Source](actions/upload-artifact@v4.4.3...v4.5.0) #### What's Changed - fix: deprecated `Node.js` version in action by [@​hamirmahal](https://github.com/hamirmahal) in [#​578](actions/upload-artifact#578) - Add new `artifact-digest` output by [@​bdehamer](https://github.com/bdehamer) in [#​656](actions/upload-artifact#656) #### New Contributors - [@​hamirmahal](https://github.com/hamirmahal) made their first contribution in [#​578](actions/upload-artifact#578) - [@​bdehamer](https://github.com/bdehamer) made their first contribution in [#​656](actions/upload-artifact#656) **Full Changelog**: <actions/upload-artifact@v4.4.3...v4.5.0> ### [`v4.4.3`](https://github.com/actions/upload-artifact/releases/tag/v4.4.3) [Compare Source](actions/upload-artifact@v4.4.2...v4.4.3) #### What's Changed - Undo indirect dependency updates from [#​627](actions/upload-artifact#627) by [@​joshmgross](https://github.com/joshmgross) in [#​632](actions/upload-artifact#632) **Full Changelog**: <actions/upload-artifact@v4.4.2...v4.4.3> ### [`v4.4.2`](https://github.com/actions/upload-artifact/releases/tag/v4.4.2) [Compare Source](actions/upload-artifact@v4.4.1...v4.4.2) #### What's Changed - Bump `@actions/artifact` to 2.1.11 by [@​robherley](https://github.com/robherley) in [#​627](actions/upload-artifact#627) - Includes fix for relative symlinks not resolving properly **Full Changelog**: <actions/upload-artifact@v4.4.1...v4.4.2> ### [`v4.4.1`](https://github.com/actions/upload-artifact/releases/tag/v4.4.1) [Compare Source](actions/upload-artifact@v4.4.0...v4.4.1) #### What's Changed - Add a section about hidden files by [@​joshmgross](https://github.com/joshmgross) in [#​607](actions/upload-artifact#607) - Add workflow file for publishing releases to immutable action package by [@​Jcambass](https://github.com/Jcambass) in [#​621](actions/upload-artifact#621) - Update [@​actions/artifact](https://github.com/actions/artifact) to latest version, includes symlink and timeout fixes by [@​robherley](https://github.com/robherley) in [#​625](actions/upload-artifact#625) #### New Contributors - [@​Jcambass](https://github.com/Jcambass) made their first contribution in [#​621](actions/upload-artifact#621) **Full Changelog**: <actions/upload-artifact@v4.4.0...v4.4.1> ### [`v4.4.0`](https://github.com/actions/upload-artifact/releases/tag/v4.4.0) [Compare Source](actions/upload-artifact@v4.3.6...v4.4.0) #### Notice: Breaking Changes⚠️ We will no longer include hidden files and folders by default in the `upload-artifact` action of this version. This reduces the risk that credentials are accidentally uploaded into artifacts. Customers who need to continue to upload these files can use a new option, `include-hidden-files`, to continue to do so. See ["Notice of upcoming deprecations and breaking changes in GitHub Actions runners"](https://github.blog/changelog/2024-08-19-notice-of-upcoming-deprecations-and-breaking-changes-in-github-actions-runners/) changelog and [this issue](actions/upload-artifact#602) for more details. #### What's Changed - Exclude hidden files by default by [@​joshmgross](https://github.com/joshmgross) in [#​598](actions/upload-artifact#598) **Full Changelog**: <actions/upload-artifact@v4.3.6...v4.4.0> ### [`v4.3.6`](https://github.com/actions/upload-artifact/releases/tag/v4.3.6) [Compare Source](actions/upload-artifact@v4.3.5...v4.3.6) #### What's Changed - Revert to [@​actions/artifact](https://github.com/actions/artifact) 2.1.8 by [@​robherley](https://github.com/robherley) in [#​594](actions/upload-artifact#594) **Full Changelog**: <actions/upload-artifact@v4...v4.3.6> ### [`v4.3.5`](https://github.com/actions/upload-artifact/releases/tag/v4.3.5) [Compare Source](actions/upload-artifact@v4.3.4...v4.3.5) #### What's Changed - Bump [@​actions/artifact](https://github.com/actions/artifact) to v2.1.9 by [@​robherley](https://github.com/robherley) in [#​588](actions/upload-artifact#588) - Fixed artifact upload chunk timeout logic [#​1774](actions/toolkit#1774) - Use lazy stream to prevent issues with open file limits [#​1771](actions/toolkit#1771) **Full Changelog**: <actions/upload-artifact@v4.3.4...v4.3.5> ### [`v4.3.4`](https://github.com/actions/upload-artifact/releases/tag/v4.3.4) [Compare Source](actions/upload-artifact@v4.3.3...v4.3.4) #### What's Changed - Update [@​actions/artifact](https://github.com/actions/artifact) version, bump dependencies by [@​robherley](https://github.com/robherley) in [#​584](actions/upload-artifact#584) **Full Changelog**: <actions/upload-artifact@v4.3.3...v4.3.4> ### [`v4.3.3`](https://github.com/actions/upload-artifact/releases/tag/v4.3.3) [Compare Source](actions/upload-artifact@v4.3.2...v4.3.3) #### What's Changed - updating `@actions/artifact` dependency to v2.1.6 by [@​eggyhead](https://github.com/eggyhead) in [#​565](actions/upload-artifact#565) **Full Changelog**: <actions/upload-artifact@v4.3.2...v4.3.3> ### [`v4.3.2`](https://github.com/actions/upload-artifact/releases/tag/v4.3.2) [Compare Source](actions/upload-artifact@v4.3.1...v4.3.2) #### What's Changed - Update release-new-action-version.yml by [@​konradpabjan](https://github.com/konradpabjan) in [#​516](actions/upload-artifact#516) - Minor fix to the migration readme by [@​andrewakim](https://github.com/andrewakim) in [#​523](actions/upload-artifact#523) - Update readme with v3/v2/v1 deprecation notice by [@​robherley](https://github.com/robherley) in [#​561](actions/upload-artifact#561) - updating `@actions/artifact` dependency to v2.1.5 and `@actions/core` to v1.0.1 by [@​eggyhead](https://github.com/eggyhead) in [#​562](actions/upload-artifact#562) #### New Contributors - [@​andrewakim](https://github.com/andrewakim) made their first contribution in [#​523](actions/upload-artifact#523) **Full Changelog**: <actions/upload-artifact@v4.3.1...v4.3.2> ### [`v4.3.1`](https://github.com/actions/upload-artifact/releases/tag/v4.3.1) [Compare Source](actions/upload-artifact@v4.3.0...v4.3.1) - Bump [@​actions/artifacts](https://github.com/actions/artifacts) to latest version to include [updated GHES host check](actions/toolkit#1648) ### [`v4.3.0`](https://github.com/actions/upload-artifact/releases/tag/v4.3.0) [Compare Source](actions/upload-artifact@v4.2.0...v4.3.0) #### What's Changed - Reorganize upload code in prep for merge logic & add more tests by [@​robherley](https://github.com/robherley) in [#​504](actions/upload-artifact#504) - Add sub-action to merge artifacts by [@​robherley](https://github.com/robherley) in [#​505](actions/upload-artifact#505) **Full Changelog**: <actions/upload-artifact@v4...v4.3.0> ### [`v4.2.0`](https://github.com/actions/upload-artifact/releases/tag/v4.2.0) [Compare Source](actions/upload-artifact@v4.1.0...v4.2.0) #### What's Changed - Ability to overwrite an Artifact by [@​robherley](https://github.com/robherley) in [#​501](actions/upload-artifact#501) **Full Changelog**: <actions/upload-artifact@v4...v4.2.0> ### [`v4.1.0`](https://github.com/actions/upload-artifact/releases/tag/v4.1.0) [Compare Source](actions/upload-artifact@v4...v4.1.0) #### What's Changed - Add migrations docs by [@​robherley](https://github.com/robherley) in [#​482](actions/upload-artifact#482) - Update README.md by [@​samuelwine](https://github.com/samuelwine) in [#​492](actions/upload-artifact#492) - Support artifact-url output by [@​konradpabjan](https://github.com/konradpabjan) in [#​496](actions/upload-artifact#496) - Update readme to reflect new 500 artifact per job limit by [@​robherley](https://github.com/robherley) in [#​497](actions/upload-artifact#497) #### New Contributors - [@​samuelwine](https://github.com/samuelwine) made their first contribution in [#​492](actions/upload-artifact#492) **Full Changelog**: <actions/upload-artifact@v4...v4.1.0> ### [`v4.0.0`](https://github.com/actions/upload-artifact/releases/tag/v4.0.0) [Compare Source](actions/upload-artifact@v4...v4) #### What's Changed The release of upload-artifact\@​v4 and download-artifact\@​v4 are major changes to the backend architecture of Artifacts. They have numerous performance and behavioral improvements. ℹ️ However, this is a major update that includes breaking changes. Artifacts created with versions v3 and below are not compatible with the v4 actions. Uploads and downloads *must* use the same major actions versions. There are also key differences from previous versions that may require updates to your workflows. For more information, please see: 1. The [changelog](https://github.blog/changelog/2023-12-14-github-actions-artifacts-v4-is-now-generally-available/) post. 2. The [README](https://github.com/actions/upload-artifact/blob/main/README.md). 3. The [migration documentation](https://github.com/actions/upload-artifact/blob/main/docs/MIGRATION.md). 4. As well as the underlying npm package, [@​actions/artifact](https://github.com/actions/toolkit/tree/main/packages/artifact) documentation. #### New Contributors - [@​vmjoseph](https://github.com/vmjoseph) made their first contribution in [#​464](actions/upload-artifact#464) **Full Changelog**: <actions/upload-artifact@v3...v4.0.0> ### [`v4`](actions/upload-artifact@v3.2.2...v4) [Compare Source](actions/upload-artifact@v3.2.2...v4) </details> --- ### Configuration 📅 **Schedule**: (UTC) - Branch creation - At any time (no schedule defined) - Automerge - At any time (no schedule defined) 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](undefined) if that's undesired. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4xNTAuMCIsInVwZGF0ZWRJblZlciI6IjQzLjIzMy4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJ0eXBlL2RlcGVuZGVuY2llcyJdfQ==--> Reviewed-on: https://git.tainton.uk/repos/epage-go/pulls/10 Co-authored-by: renovate[bot] <renovate-bot@git.tainton.uk> Co-committed-by: renovate[bot] <renovate-bot@git.tainton.uk>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
While looking at CI output for:
We had an issue/race where we were trying to do stuff after the test completed:
Turns out we had a few errors in our tests and I also discovered we were not handling our timeout logic correctly:
The problematic area of the timeout logic was that it was throwing an error within the
setTimeoutblock, which is outside the promise area and went unhandled. To remedy this (and avoid other leaky async code) I swapped it with asetIntervalwrapped in a Promise that would check the last chunk upload. Then, I gave this function and the blob upload anAbortControllerso we can quit everything when we want to. Then, the main await is aPromise.racewhere we always abort after the first promise resolves.As for the tests, we were incorrectly mocking over the logic we were testing and throwing immediately. I also took the liberty of updating the rest of the tests since there was a lot of duplication that made each test harder to reason about.