robojerk/codeql-action

Add more invalid SARIF processing errors

Browse source
This commit is contained in:
Henry Mercer 2024-05-08 13:36:10 +01:00
parent 45fb58ab31
commit b1c3187939
3 changed files with 5 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
lib/upload-lib.js generated
View file

@ -520,6 +520,8 @@ function shouldConsiderConfigurationError(processingErrors) {
*/ */
function shouldConsiderInvalidRequest(processingErrors) { function shouldConsiderInvalidRequest(processingErrors) {
return processingErrors.every((error) => error.startsWith("rejecting SARIF") || return processingErrors.every((error) => error.startsWith("rejecting SARIF") ||
error.startsWith("an invalid URI was provided as a SARIF location") ||
error.startsWith("locationFromSarifResult: expected artifact location") ||
error.startsWith("could not convert rules: invalid security severity value, is not a number") || error.startsWith("could not convert rules: invalid security severity value, is not a number") ||
/^SARIF URI scheme [^\s]* did not match the checkout URI scheme [^\s]*/.test(error)); /^SARIF URI scheme [^\s]* did not match the checkout URI scheme [^\s]*/.test(error));
} }

2
lib/upload-lib.js.map
View file

File diff suppressed because one or more lines are too long

2
src/upload-lib.ts
View file

@ -775,6 +775,8 @@ function shouldConsiderInvalidRequest(processingErrors: string[]): boolean {
return processingErrors.every( return processingErrors.every(
(error) => (error) =>
error.startsWith("rejecting SARIF") || error.startsWith("rejecting SARIF") ||
error.startsWith("an invalid URI was provided as a SARIF location") ||
error.startsWith("locationFromSarifResult: expected artifact location") ||
error.startsWith( error.startsWith(
"could not convert rules: invalid security severity value, is not a number", "could not convert rules: invalid security severity value, is not a number",
) || ) ||