diff --git a/src/vs/platform/environment/common/argv.ts b/src/vs/platform/environment/common/argv.ts index 8420a50ebb9c92424ffacfcad093823de36945b0..70a7eb4b86e4188b418300e8929808641eb24fcc 100644 --- a/src/vs/platform/environment/common/argv.ts +++ b/src/vs/platform/environment/common/argv.ts @@ -88,6 +88,7 @@ export interface NativeParsedArgs { // chromium command line args: https://electronjs.org/docs/all#supported-chrome-command-line-switches 'no-proxy-server'?: boolean; + 'no-sandbox'?: boolean; 'proxy-server'?: string; 'proxy-bypass-list'?: string; 'proxy-pac-url'?: string; diff --git a/src/vs/platform/environment/node/argv.ts b/src/vs/platform/environment/node/argv.ts index aafba8a6b3dbe3c9bc981a69bdf859260b8f770d..2715617e2bc924caaebcf8a218e943c747addaf8 100644 --- a/src/vs/platform/environment/node/argv.ts +++ b/src/vs/platform/environment/node/argv.ts @@ -123,6 +123,12 @@ export const OPTIONS: OptionDescriptions> = { // chromium flags 'no-proxy-server': { type: 'boolean' }, + // Minimist incorrectly parses keys that start with `--no` + // https://github.com/substack/minimist/blob/aeb3e27dae0412de5c0494e9563a5f10c82cc7a9/index.js#L118-L121 + // If --no-sandbox is passed via cli wrapper it will be treated as --sandbox which is incorrect, we use + // the alias here to make sure --no-sandbox is always respected. + // For https://github.com/microsoft/vscode/issues/128279 + 'no-sandbox': { type: 'boolean', alias: 'sandbox' }, 'proxy-server': { type: 'string' }, 'proxy-bypass-list': { type: 'string' }, 'proxy-pac-url': { type: 'string' },