feat: cleanup destination directory if we fail during streaming of th…

…e extraction
github · Dec 9, 2024 · 4c20d4f · 4c20d4f
1 parent 51e71f8
commit 4c20d4f
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/lib/tools-download.js b/lib/tools-download.js
diff --git a/lib/tools-download.js.map b/lib/tools-download.js.map
diff --git a/src/tools-download.ts b/src/tools-download.ts
@@ -127,7 +127,11 @@ export async function downloadAndExtract(
     core.warning(
       `Failed to download and extract CodeQL bundle using streaming. Falling back to downloading the bundle before extracting.`,
     );
-    core.warning(getErrorMessage(e));
+    core.warning(`Error: ${getErrorMessage(e)}`);
+
+    // If we failed during processing, we want to clean up the destination directory
+    // before we try again.
+    await cleanUpGlob(dest, "CodeQL bundle", logger);
   }
 
   const toolsDownloadStart = performance.now();