lib: use internal/options to query --abort-on-uncaught-exception

Instead of using
`internalBinding('config').shouldAbortOnUncaughtException`.
Also removes that property from the binding.

PR-URL: #25862
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gus Caplan <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>

Author: joyeecheung

lib/internal/async_hooks.js

@@ -4,6 +4,11 @@ const {
   ERR_ASYNC_TYPE,
   ERR_INVALID_ASYNC_ID
 } = require('internal/errors').codes;
+
+const { getOptionValue } = require('internal/options');
+const shouldAbortOnUncaughtException =
+  getOptionValue('--abort-on-uncaught-exception');
+
 const async_wrap = internalBinding('async_wrap');
 /* async_hook_fields is a Uint32Array wrapping the uint32_t array of
  * Environment::AsyncHooks::fields_[]. Each index tracks the number of active
@@ -107,7 +112,7 @@ function fatalError(e) {
     Error.captureStackTrace(o, fatalError);
     process._rawDebug(o.stack);
   }
-  if (internalBinding('config').shouldAbortOnUncaughtException) {
+  if (shouldAbortOnUncaughtException) {
     process.abort();
   }
   process.exit(1);

lib/internal/policy/manifest.js

@@ -20,7 +20,9 @@ const { entries } = Object;
 const kIntegrities = new SafeWeakMap();
 const kReactions = new SafeWeakMap();
 const kRelativeURLStringPattern = /^\.{0,2}\//;
-const { shouldAbortOnUncaughtException } = internalBinding('config');
+const { getOptionValue } = require('internal/options');
+const shouldAbortOnUncaughtException =
+  getOptionValue('--abort-on-uncaught-exception');
 const { abort, exit, _rawDebug } = process;
 
 function REACTION_THROW(error) {

src/node_config.cc

@@ -95,9 +95,6 @@ static void Initialize(Local<Object> target,
   if (env->options()->expose_internals)
     READONLY_TRUE_PROPERTY(target, "exposeInternals");
 
-  if (env->abort_on_uncaught_exception())
-    READONLY_TRUE_PROPERTY(target, "shouldAbortOnUncaughtException");
-
   READONLY_PROPERTY(target,
                     "bits",
                     Number::New(env->isolate(), 8 * sizeof(intptr_t)));

src/node_options.cc

@@ -522,7 +522,14 @@ void GetOptions(const FunctionCallbackInfo<Value>& args) {
     switch (option_info.type) {
       case kNoOp:
       case kV8Option:
-        value = Undefined(isolate);
+        // Special case for --abort-on-uncaught-exception which is also
+        // respected by Node.js internals
+        if (item.first == "--abort-on-uncaught-exception") {
+          value = Boolean::New(
+            isolate, original_per_env->abort_on_uncaught_exception);
+        } else {
+          value = Undefined(isolate);
+        }
         break;
       case kBoolean:
         value = Boolean::New(isolate, *parser.Lookup<bool>(field, opts));