Fix 1 test and silence CodeQL warning (not introduced by this PR)

This commit is contained in:
Alexander Farber 2025-12-12 14:02:50 +01:00
parent a8ccd7b6fb
commit 9adad2f369
No known key found for this signature in database
3 changed files with 11 additions and 3 deletions

View file

@ -291,6 +291,8 @@ export async function start_sandbox(
sandboxEnv['NO_PROXY'] = noProxy;
sandboxEnv['no_proxy'] = noProxy;
}
// CLI tool intentionally executes user-provided proxy commands
// codeql-disable-next-line js/shell-command-injection-from-environment
proxyProcess = spawn('bash', ['-c', proxyCommand], {
stdio: ['ignore', 'pipe', 'pipe'],
detached: true,
@ -785,6 +787,8 @@ export async function start_sandbox(
const proxyShellArgs = isWindows
? ['/c', proxyContainerCommand]
: ['-c', proxyContainerCommand];
// CLI tool intentionally executes user-provided proxy commands in container
// codeql-disable-next-line js/shell-command-injection-from-environment
proxyProcess = spawn(proxyShell, proxyShellArgs, {
stdio: ['ignore', 'pipe', 'pipe'],
detached: true,