robojerk/codeql-action

Move database bundling to inside the try-catch

Browse source
This commit is contained in:
Robert 2023-01-06 15:28:25 +00:00
parent 6ce923c375
commit e8f7169839
3 changed files with 12 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

11
src/database-upload.ts
View file

@ -36,12 +36,13 @@ export async function uploadDatabases(
const codeql = await getCodeQL(config.codeQLCmd);
for (const language of config.languages) {
// Upload the database bundle.
// Although we are uploading arbitrary file contents to the API, it's worth
// noting that it's the API's job to validate that the contents is acceptable.
// This API method is available to anyone with write access to the repo.
const bundledDb = await bundleDb(config, language, codeql, language);
try {
// Upload the database bundle.
// Although we are uploading arbitrary file contents to the API, it's worth
// noting that it's the API's job to validate that the contents is acceptable.
// This API method is available to anyone with write access to the repo.
const bundledDb = await bundleDb(config, language, codeql, language);
await client.request(
`POST https://uploads.github.com/repos/:owner/:repo/code-scanning/codeql/databases/:language?name=:name`,
{