domain: pass opts to EventEmitter.init

PR-URL: #41414 Fixes: #41391 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]> Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
nodejs · Mar 14, 2022 · 227dea8 · 227dea8
1 parent bd71706
commit 227dea8
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 2 deletions.
diff --git a/lib/domain.js b/lib/domain.js
@@ -446,7 +446,7 @@ Domain.prototype.bind = function(cb) {
 EventEmitter.usingDomains = true;
 
 const eventInit = EventEmitter.init;
-EventEmitter.init = function() {
+EventEmitter.init = function(opts) {
   ObjectDefineProperty(this, 'domain', {
     configurable: true,
     enumerable: false,
@@ -457,7 +457,7 @@ EventEmitter.init = function() {
     this.domain = exports.active;
   }
 
-  return FunctionPrototypeCall(eventInit, this);
+  return FunctionPrototypeCall(eventInit, this, opts);
 };
 
 const eventEmit = EventEmitter.prototype.emit;

diff --git a/lib/events.js b/lib/events.js
@@ -321,6 +321,8 @@ EventEmitter.setMaxListeners =
     }
   };
 
+// If you're updating this function definition, please also update any
+// re-definitions, such as the one in the Domain module (lib/domain.js).
 EventEmitter.init = function(opts) {
 
   if (this._events === undefined ||

diff --git a/test/parallel/test-domain-ee.js b/test/parallel/test-domain-ee.js
@@ -18,3 +18,11 @@ d.on('error', common.mustCall((err) => {
 
 d.add(e);
 e.emit('error', new Error('foobar'));
+
+{
+  // Ensure initial params pass to origin `EventEmitter.init` function
+  const e = new EventEmitter({ captureRejections: true });
+  const kCapture = Object.getOwnPropertySymbols(e)
+    .find((it) => it.description === 'kCapture');
+  assert.strictEqual(e[kCapture], true);
+}