do not auto-enable container sandboxing (fixing recently introduced bug) (#939)
This commit is contained in:
parent
7ba2b13870
commit
e2d689ff2f
|
@ -134,11 +134,12 @@ export function sandbox_command(sandbox?: string | boolean): string {
|
||||||
}
|
}
|
||||||
|
|
||||||
// look for seatbelt, docker, or podman, in that order
|
// look for seatbelt, docker, or podman, in that order
|
||||||
|
// for container-based sandboxing, require sandbox to be enabled explicitly
|
||||||
if (os.platform() === 'darwin' && commandExists.sync('sandbox-exec')) {
|
if (os.platform() === 'darwin' && commandExists.sync('sandbox-exec')) {
|
||||||
return 'sandbox-exec';
|
return 'sandbox-exec';
|
||||||
} else if (commandExists.sync('docker')) {
|
} else if (commandExists.sync('docker') && sandbox === true) {
|
||||||
return 'docker';
|
return 'docker';
|
||||||
} else if (commandExists.sync('podman')) {
|
} else if (commandExists.sync('podman') && sandbox === true) {
|
||||||
return 'podman';
|
return 'podman';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue