From 523ce8ccda300661be06e592b615399fcfc282dc Mon Sep 17 00:00:00 2001 From: Andrei Kashchikhin Date: Mon, 1 Sep 2025 11:52:11 +0200 Subject: [PATCH 1/3] add reject --- packages/artifact/src/internal/download/download-artifact.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/packages/artifact/src/internal/download/download-artifact.ts b/packages/artifact/src/internal/download/download-artifact.ts index 4a735bb8..1616aff1 100644 --- a/packages/artifact/src/internal/download/download-artifact.ts +++ b/packages/artifact/src/internal/download/download-artifact.ts @@ -82,6 +82,7 @@ export async function streamExtractExternal( response.message.destroy( new Error(`Blob storage chunk did not respond in ${timeout}ms`) ) + reject(`Blob storage chunk did not respond in ${timeout}ms`) } const timer = setTimeout(timerFn, timeout) From 7c689a5156668994dd39e3b783380a224c0987cb Mon Sep 17 00:00:00 2001 From: Andrei Kashchikhin Date: Wed, 24 Sep 2025 17:05:25 +0200 Subject: [PATCH 2/3] use error in both reject and destroy --- .../artifact/src/internal/download/download-artifact.ts | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/packages/artifact/src/internal/download/download-artifact.ts b/packages/artifact/src/internal/download/download-artifact.ts index 1616aff1..d001a2f8 100644 --- a/packages/artifact/src/internal/download/download-artifact.ts +++ b/packages/artifact/src/internal/download/download-artifact.ts @@ -79,10 +79,9 @@ export async function streamExtractExternal( return new Promise((resolve, reject) => { const timerFn = (): void => { - response.message.destroy( - new Error(`Blob storage chunk did not respond in ${timeout}ms`) - ) - reject(`Blob storage chunk did not respond in ${timeout}ms`) + const timeoutError = new Error(`Blob storage chunk did not respond in ${timeout}ms`) + response.message.destroy(timeoutError) + reject(timeoutError) } const timer = setTimeout(timerFn, timeout) From 844423665b414e179561457ff33f81b4f794c0c2 Mon Sep 17 00:00:00 2001 From: Andrei Kashchikhin Date: Thu, 25 Sep 2025 10:53:34 +0200 Subject: [PATCH 3/3] lint --- packages/artifact/src/internal/download/download-artifact.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packages/artifact/src/internal/download/download-artifact.ts b/packages/artifact/src/internal/download/download-artifact.ts index d001a2f8..090260f9 100644 --- a/packages/artifact/src/internal/download/download-artifact.ts +++ b/packages/artifact/src/internal/download/download-artifact.ts @@ -79,7 +79,9 @@ export async function streamExtractExternal( return new Promise((resolve, reject) => { const timerFn = (): void => { - const timeoutError = new Error(`Blob storage chunk did not respond in ${timeout}ms`) + const timeoutError = new Error( + `Blob storage chunk did not respond in ${timeout}ms` + ) response.message.destroy(timeoutError) reject(timeoutError) }