From 2efff809ebd2106e62b39a2cfba3a09fca10161a Mon Sep 17 00:00:00 2001 From: Koen Vlaswinkel Date: Wed, 28 Sep 2022 13:18:47 +0200 Subject: [PATCH] Fix not being able to run variant analyses The `controllerRepo` parameter was being encoded/escaped by Octokit, resulting in a URL like `repos/dsp-testing%2Fqc-controller/code-scanning/codeql/queries` rather than `repos/dsp-testing/qc-controller/code-scanning/codeql/queries`. This switches it to use the ID instead, since we already have the ID and do not have access to the owner and repo separately anymore. --- extensions/ql-vscode/src/remote-queries/run-remote-query.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/extensions/ql-vscode/src/remote-queries/run-remote-query.ts b/extensions/ql-vscode/src/remote-queries/run-remote-query.ts index 0415713e0..bc1b02fbc 100644 --- a/extensions/ql-vscode/src/remote-queries/run-remote-query.ts +++ b/extensions/ql-vscode/src/remote-queries/run-remote-query.ts @@ -361,9 +361,9 @@ async function runRemoteQueriesApiRequest( try { const octokit = await credentials.getOctokit(); const response: OctokitResponse = await octokit.request( - 'POST /repos/:controllerRepo/code-scanning/codeql/queries', + 'POST /repositories/:controllerRepoId/code-scanning/codeql/queries', { - controllerRepo: controllerRepo.fullName, + controllerRepoId: controllerRepo.id, data } );