lib: make process.binding('util') return only type checkers

Ref: #37485 (review) Ref: #37787 PR-URL: #37819 Refs: #37787 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Anto Aravinth <[email protected]>
nodejs · Mar 27, 2021 · fe73e4d · fe73e4d
1 parent 924238d
commit fe73e4d
Show file tree

Hide file tree

Showing 4 changed files with 74 additions and 0 deletions.
diff --git a/lib/internal/bootstrap/loaders.js b/lib/internal/bootstrap/loaders.js
@@ -113,6 +113,10 @@ const runtimeDeprecatedList = new SafeSet([
   'v8',
 ]);
 
+const legacyWrapperList = new SafeSet([
+  'util',
+]);
+
 // Set up process.binding() and process._linkedBinding().
 {
   const bindingObj = ObjectCreate(null);
@@ -129,6 +133,9 @@ const runtimeDeprecatedList = new SafeSet([
           'DeprecationWarning',
           'DEP0111');
       }
+      if (legacyWrapperList.has(module)) {
+        return nativeModuleRequire('internal/legacy/processbinding')[module]();
+      }
       return internalBinding(module);
     }
     // eslint-disable-next-line no-restricted-syntax

diff --git a/lib/internal/legacy/processbinding.js b/lib/internal/legacy/processbinding.js
@@ -0,0 +1,36 @@
+'use strict';
+const {
+  ArrayPrototypeFilter,
+  ArrayPrototypeIncludes,
+  ObjectFromEntries,
+  ObjectEntries,
+  SafeArrayIterator,
+} = primordials;
+const { types } = require('util');
+
+module.exports = {
+  util() {
+    return ObjectFromEntries(new SafeArrayIterator(ArrayPrototypeFilter(
+      ObjectEntries(types),
+      ({ 0: key }) => {
+        return ArrayPrototypeIncludes([
+          'isArrayBuffer',
+          'isArrayBufferView',
+          'isAsyncFunction',
+          'isDataView',
+          'isDate',
+          'isExternal',
+          'isMap',
+          'isMapIterator',
+          'isNativeError',
+          'isPromise',
+          'isRegExp',
+          'isSet',
+          'isSetIterator',
+          'isTypedArray',
+          'isUint8Array',
+          'isAnyArrayBuffer',
+        ], key);
+      })));
+  }
+};
diff --git a/node.gyp b/node.gyp
@@ -168,6 +168,7 @@
       'lib/internal/idna.js',
       'lib/internal/inspector_async_hook.js',
       'lib/internal/js_stream_socket.js',
+      'lib/internal/legacy/processbinding.js',
       'lib/internal/linkedlist.js',
       'lib/internal/main/check_syntax.js',
       'lib/internal/main/eval_string.js',

diff --git a/test/parallel/test-process-binding-util.js b/test/parallel/test-process-binding-util.js
@@ -0,0 +1,30 @@
+'use strict';
+require('../common');
+const assert = require('assert');
+const util = require('util');
+
+const utilBinding = process.binding('util');
+assert.deepStrictEqual(
+  Object.keys(utilBinding).sort(),
+  [
+    'isAnyArrayBuffer',
+    'isArrayBuffer',
+    'isArrayBufferView',
+    'isAsyncFunction',
+    'isDataView',
+    'isDate',
+    'isExternal',
+    'isMap',
+    'isMapIterator',
+    'isNativeError',
+    'isPromise',
+    'isRegExp',
+    'isSet',
+    'isSetIterator',
+    'isTypedArray',
+    'isUint8Array',
+  ]);
+
+for (const k of Object.keys(utilBinding)) {
+  assert.strictEqual(utilBinding[k], util.types[k]);
+}