[tech-debt] sigv4/S3 client duplicated across artifact+cache actions and has drifted #11

Open
opened 2026-06-23 20:14:55 +00:00 by stephen · 0 comments
Owner

Autonomous audit finding (MED).

backend.js/sigv4.js/core.js are byte-identical across upload/download/cache actions, but backends/s3.js has drifted into 3 different versions (different remove()/listTree()), and glob.js/artifact-tar.js have drifted too. The signed-HTTP S3 client is maintained by copy-paste. Extract a shared package, or add a vendored known-answer sigv4 test in each repo to catch drift. (sigv4/s3 are only tested in cache-action today.)

Autonomous audit finding (MED). `backend.js`/`sigv4.js`/`core.js` are byte-identical across upload/download/cache actions, but `backends/s3.js` has drifted into 3 different versions (different `remove()`/`listTree()`), and `glob.js`/`artifact-tar.js` have drifted too. The signed-HTTP S3 client is maintained by copy-paste. Extract a shared package, or add a vendored known-answer sigv4 test in each repo to catch drift. (sigv4/s3 are only tested in cache-action today.)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rasterstate/download-artifact-action#11
No description provided.