stream: always delay construct callback by a nextTick

Signed-off-by: Matteo Collina <hello@matteocollina.com> Fixes: #46765
nodejs · Feb 24, 2023 · db2554c · db2554c
1 parent b6c56cb
commit db2554c
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 2 deletions.
diff --git a/lib/internal/streams/destroy.js b/lib/internal/streams/destroy.js
@@ -271,10 +271,14 @@ function constructNT(stream) {
     }
   }
 
+  function jump (err) {
+    process.nextTick(onConstruct, err);
+  }
+
   try {
-    stream._construct(onConstruct);
+    stream._construct(jump);
   } catch (err) {
-    onConstruct(err);
+    jump(err);
   }
 }
 

diff --git a/test/parallel/test-stream2-transform.js b/test/parallel/test-stream2-transform.js
@@ -468,3 +468,27 @@ const { PassThrough, Transform } = require('stream');
     assert.strictEqual(ended, true);
   }));
 }
+
+{
+  const s = new Transform({
+    objectMode: true,
+    construct(callback) {
+      this.push('header from constructor');
+      callback();
+    },
+    transform: (row, encoding, callback) => {
+      callback(null, row);
+    },
+  });
+
+  const expected = [
+    'header from constructor',
+    'firstLine',
+    'secondLine'
+  ]
+  s.on('data', common.mustCall((data) => {
+    assert.strictEqual(data.toString(), expected.shift());
+  }, 3));
+  s.write('firstLine');
+  process.nextTick(() => s.write('secondLine'));
+}