From d7543a79d6949213950a60393d3bc16d18a6ae29 Mon Sep 17 00:00:00 2001
From: Joyee Cheung <joyeec9h3@gmail.com>
Date: Wed, 6 Mar 2019 12:05:23 +0100
Subject: [PATCH] process: set up process warning handler in pre-execution

Since it depends on environment variables.

PR-URL: https://github.com/nodejs/node/pull/26466
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
---
 lib/internal/bootstrap/node.js          |  5 +----
 lib/internal/bootstrap/pre_execution.js | 12 ++++++++++++
 lib/internal/main/worker_thread.js      |  3 +++
 3 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/lib/internal/bootstrap/node.js b/lib/internal/bootstrap/node.js
index aa6c6b262a4c2d..f04bfe23fbc3f6 100644
--- a/lib/internal/bootstrap/node.js
+++ b/lib/internal/bootstrap/node.js
@@ -115,12 +115,9 @@ if (isMainThread) {
 }
 
 const {
-  onWarning,
   emitWarning
 } = NativeModule.require('internal/process/warning');
-if (!process.noProcessWarnings && process.env.NODE_NO_WARNINGS !== '1') {
-  process.on('warning', onWarning);
-}
+
 process.emitWarning = emitWarning;
 
 const {
diff --git a/lib/internal/bootstrap/pre_execution.js b/lib/internal/bootstrap/pre_execution.js
index 1ba6c7a7580f49..9dc93a54780b65 100644
--- a/lib/internal/bootstrap/pre_execution.js
+++ b/lib/internal/bootstrap/pre_execution.js
@@ -8,6 +8,8 @@ let traceEventsAsyncHook;
 function prepareMainThreadExecution() {
   setupTraceCategoryState();
 
+  setupWarningHandler();
+
   // Only main thread receives signals.
   setupSignalHandlers();
 
@@ -36,6 +38,15 @@ function prepareMainThreadExecution() {
   loadPreloadModules();
 }
 
+function setupWarningHandler() {
+  const {
+    onWarning
+  } = require('internal/process/warning');
+  if (!process.noProcessWarnings && process.env.NODE_NO_WARNINGS !== '1') {
+    process.on('warning', onWarning);
+  }
+}
+
 function initializeReport() {
   if (!getOptionValue('--experimental-report')) {
     return;
@@ -245,6 +256,7 @@ function loadPreloadModules() {
 }
 
 module.exports = {
+  setupWarningHandler,
   prepareMainThreadExecution,
   initializeDeprecations,
   initializeESMLoader,
diff --git a/lib/internal/main/worker_thread.js b/lib/internal/main/worker_thread.js
index 0dc1b61ef94f45..4ecceff4e6ec41 100644
--- a/lib/internal/main/worker_thread.js
+++ b/lib/internal/main/worker_thread.js
@@ -4,6 +4,7 @@
 // message port.
 
 const {
+  setupWarningHandler,
   initializeDeprecations,
   initializeESMLoader,
   initializeFrozenIntrinsics,
@@ -39,6 +40,8 @@ const {
 const publicWorker = require('worker_threads');
 const debug = require('util').debuglog('worker');
 
+setupWarningHandler();
+
 debug(`[${threadId}] is setting up worker child environment`);
 
 // Set up the message port and start listening