Use repository NWO owner instead

github · Jan 12, 2024 · a0afcce · a0afcce
1 parent 4885993
commit a0afcce
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 3 deletions.
diff --git a/lib/init-action-post-helper.js b/lib/init-action-post-helper.js
diff --git a/lib/init-action-post-helper.js.map b/lib/init-action-post-helper.js.map
diff --git a/src/init-action-post-helper.ts b/src/init-action-post-helper.ts
@@ -185,7 +185,12 @@ export async function run(
   }
 
   core.info(`GITHUB_ACTOR is ${process.env["GITHUB_ACTOR"]}`);
-  if (process.env["CODEQL_ACTION_EXPECT_UPLOAD_FAILED_SARIF"] === "true") {
+  // We do not delete uploaded SARIFs if we're on a fork, as we're missing the
+  // appropriate permissions.
+  if (
+    process.env["CODEQL_ACTION_EXPECT_UPLOAD_FAILED_SARIF"] === "true" &&
+    repositoryNwo.owner !== "github"
+  ) {
     await removeUploadedSarif(uploadFailedSarifResult, logger);
   }