b43ff0bde6
Migrate artifact package from @azure/core-http to @azure/core-rest-pipeline
...
Co-authored-by: salmanmkc <32169182+salmanmkc@users.noreply.github.com >
2025-12-10 23:54:25 +00:00
02afeb1577
style: wrap ACTIONS_ARTIFACT_MAX_ARTIFACT_COUNT env var assignment for readability
2025-10-22 11:42:01 -04:00
cbc06d6766
fix: ensure max artifact count variable is treated as a string
2025-10-22 07:47:51 -04:00
9bb6708527
fix: remove redundant check for max artifact count variable
2025-10-22 07:41:31 -04:00
be1151df02
Apply suggestion from @Link-
...
Co-authored-by: Bassem Dghaidi <568794+Link-@users.noreply.github.com >
2025-10-22 07:39:45 -04:00
fea4f6b5c5
fix: resolve critical pagination bugs and add comprehensive testing
...
- Fix off-by-one error in pagination loop (< to <=) that prevented fetching last page
- Add Math.ceil() to maxNumberOfPages calculation for proper limit handling
- Replace hardcoded 2000 limit with configurable getMaxArtifactListCount()
- Add pagination test for multi-page artifact listing
- Add environment variable test for ACTIONS_ARTIFACT_MAX_ARTIFACT_COUNT
- Add comprehensive test coverage for getMaxArtifactListCount() function
Fixes compound bug where pagination and limit logic capped results at 900 artifacts instead of intended 1000.
2025-10-21 09:22:11 -04:00
fb592eec03
Update packages/artifact/src/internal/find/list-artifacts.ts
...
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com >
2025-10-17 18:05:06 -04:00
70e79399a2
fix: bumping max list artifact to 2k
2025-10-16 23:13:34 -04:00
b8ac8fc14a
lint
2025-10-15 17:08:33 +01:00
028d621193
Merge remote-tracking branch 'origin/main' into salmanmkc/node24
2025-10-15 16:41:54 +01:00
b0d901f9c2
rebase led to this changing so reverting
2025-10-15 16:37:38 +01:00
33a9b6c09c
update with dist updates
2025-10-15 16:22:51 +01:00
9b08f07cd3
Fix linting
2025-09-25 09:26:13 -04:00
d26e9423f4
Test: add a timeout test for downloading chunks from the stream
2025-09-25 09:11:38 -04:00
844423665b
lint
2025-09-25 10:53:34 +02:00
7c689a5156
use error in both reject and destroy
2025-09-24 17:05:25 +02:00
523ce8ccda
add reject
2025-09-01 11:52:11 +02:00
2559a2ac8a
Merge pull request #1982 from actions/salmanmkc/obfuscate-sas
...
Remove logging of any SAS tokens in Actions/Cache and Actions/Artifact
2025-03-17 11:47:29 +00:00
957d42e6c5
add encoding back with extra tests
2025-03-14 06:38:57 -07:00
39419dd8c3
don't need to url encode or set var
2025-03-14 06:21:41 -07:00
6876e2a664
update ts docs
2025-03-13 04:47:49 -07:00
fc482662af
PR feedback, back to simplified approach, no export on client as well
2025-03-13 04:23:45 -07:00
3ac34ffcb7
Mask different situations, malformed URL, encoded, decoded, raw signatures, nested parameters, and moved to a utility file
2025-03-12 03:17:35 -07:00
56c5a39afb
Update blob-upload.ts
2025-03-12 07:59:00 +00:00
d7ddca4309
Fix comment on expectedHash
2025-03-11 10:52:19 +00:00
47c4fa85df
masks the whole URL, update tests
2025-03-10 06:47:52 -07:00
1cd2f8a538
Instead of using utility method in core lib, use method in both twirp clients
2025-03-07 06:01:25 -08:00
dc22dc7cad
Merge pull request #1975 from actions/ghadimir/update_call_to_list_artifacts
...
Compare Artifact Digests
2025-03-07 09:51:05 +00:00
8c05dc87d8
Change info logs to debug logs
2025-03-07 09:38:33 +00:00
944e6b78db
Add secret and signature masking for cache and artifact packages
2025-03-06 14:25:32 -08:00
3726c11433
Please the linter
2025-03-05 14:44:58 +00:00
71b40f7024
nicer wording
2025-03-05 14:35:01 +00:00
83e5e2517b
Change some debug -> info for artifacts hash logging
2025-03-05 14:30:51 +00:00
d5c8a0fa27
Update proto artifact interface, retrieve artifact digests, return indicator of mismatch failure
2025-03-05 11:29:44 +00:00
780e24be34
Dont skip pages
2025-03-05 09:27:35 +00:00
c26e6f3aba
Default upload artifacts concurrency to 5
2025-02-20 17:03:29 +00:00
e0c069db55
remove runtime dependency on twirp-ts
2025-01-27 17:52:55 +00:00
e55409315f
Rename the prefix to be more specific
2025-01-08 20:32:45 +00:00
d4385a64a7
Concurrency has a min of 1
2025-01-08 18:14:04 +00:00
ede05b95d7
Make concurrency change opt-in, but can only go lower
2025-01-08 18:11:38 +00:00
f3c12d5561
Set default concurrency to 10 and make timeout configurable
2025-01-08 16:19:09 +00:00
1e0c16f0dc
return artifact digest on upload
...
Signed-off-by: Brian DeHamer <bdehamer@github.com >
2024-12-06 14:27:46 -08:00
545e0e6b95
properly resolve relative symlinks
2024-10-08 12:35:48 -04:00
7f19a7886a
fix regression, auto readlink on symlinks again
2024-09-20 17:23:43 -04:00
3a33cca851
FIX: Set chunk timeout back to 5 minutes
2024-08-06 10:27:41 +02:00
7c61054649
Remove unused import
2024-07-27 17:00:02 +07:00
9517cdf52d
Prevent "too many open files" in artifact upload
...
See https://www.archiverjs.com/docs/archiver/#file
2024-07-26 08:49:34 +07:00
3e34f6d19c
add comment for chunk timeout
2024-07-24 12:40:57 -04:00
182702d2df
fix chunk timeout + update tests
2024-07-23 21:57:39 -04:00
176b40a888
allow localhost hostnames for artifact checks
2024-07-03 16:55:53 +00:00
copilot-swe-agent[bot]