inspector: refactor inspector/promises to be more robust

Limit the effect of users mutating `node:util`, `node:inspector`, or `Array.prototype`. PR-URL: #45041 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Nov 1, 2022 · 6f6a8ce · 6f6a8ce
1 parent 4f7f228
commit 6f6a8ce
Showing 1 changed file with 4 additions and 12 deletions.
diff --git a/lib/inspector/promises.js b/lib/inspector/promises.js
@@ -1,20 +1,12 @@
 'use strict';
 
 const inspector = require('inspector');
-const { promisify } = require('util');
-const { FunctionPrototypeBind } = primordials;
+const { promisify } = require('internal/util');
+
 class Session extends inspector.Session {
-  #post = promisify(FunctionPrototypeBind(super.post, this));
-  /**
-   * Posts a message to the inspector back-end.
-   * @param {string} method
-   * @param {Record<unknown, unknown>} [params]
-   * @returns {Promise}
-   */
-  async post(method, params) {
-    return this.#post(method, params);
-  }
+  constructor() { super(); } // eslint-disable-line no-useless-constructor
 }
+Session.prototype.post = promisify(inspector.Session.prototype.post);
 
 module.exports = {
   ...inspector,