diff options
author | Raymond Zhao <rzhao271@gmail.com> | 2022-02-11 20:30:56 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-02-11 20:30:56 +0300 |
commit | 863d13c192a8d315fa274194e64c1c9e5820e8f2 (patch) | |
tree | fd001222440c8defc13b1d2df7d8d7543fa1d282 /src/spawn_sync.cc | |
parent | 998e5acc69b92ef39e38db133b9bfb8bb7af7e82 (diff) |
child_process: kHideConsoleWindows for spawnSync
kHideConsoleWindows should hide console windows for
execSync and spawnSync calls as well. This fix adds on
to https://github.com/nodejs/node/pull/39712.
PR-URL: https://github.com/nodejs/node/pull/41412
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'src/spawn_sync.cc')
-rw-r--r-- | src/spawn_sync.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/spawn_sync.cc b/src/spawn_sync.cc index 1141aceae98..afd08519d7f 100644 --- a/src/spawn_sync.cc +++ b/src/spawn_sync.cc @@ -810,6 +810,9 @@ Maybe<int> SyncProcessRunner::ParseOptions(Local<Value> js_value) { if (js_win_hide->BooleanValue(isolate)) uv_process_options_.flags |= UV_PROCESS_WINDOWS_HIDE; + if (env()->hide_console_windows()) + uv_process_options_.flags |= UV_PROCESS_WINDOWS_HIDE_CONSOLE; + Local<Value> js_wva = js_options->Get(context, env()->windows_verbatim_arguments_string()) .ToLocalChecked(); |