child_process: kHideConsoleWindows for spawnSync

kHideConsoleWindows should hide console windows for execSync and spawnSync calls as well. This fix adds on to #39712. PR-URL: #41412 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Feb 22, 2022 · 24b0a56 · 24b0a56
1 parent 3a12134
commit 24b0a56
Showing 1 changed file with 3 additions and 0 deletions.
diff --git 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();