child_process: fire close event from stdio

PR-URL: #22892
Reviewed-By: Anna Henningsen <[email protected]>

Author: koh110

lib/internal/child_process.js

@@ -386,6 +386,9 @@ ChildProcess.prototype.spawn = function(options) {
     // The stream is already cloned and piped, thus close it.
     if (stream.type === 'wrap') {
       stream.handle.close();
+      if (stream._stdio && stream._stdio instanceof EventEmitter) {
+        stream._stdio.emit('close');
+      }
       continue;
     }
 
@@ -946,7 +949,8 @@ function _validateStdio(stdio, sync) {
       acc.push({
         type: 'wrap',
         wrapType: getHandleWrapType(handle),
-        handle: handle
+        handle: handle,
+        _stdio: stdio
       });
     } else if (isArrayBufferView(stdio) || typeof stdio === 'string') {
       if (!sync) {

test/parallel/test-child-process-server-close.js

@@ -0,0 +1,20 @@
+'use strict';
+
+const common = require('../common');
+const { spawn } = require('child_process');
+const net = require('net');
+
+const server = net.createServer((conn) => {
+  conn.on('close', common.mustCall());
+
+  spawn(process.execPath, ['-v'], {
+    stdio: ['ignore', conn, 'ignore']
+  }).on('close', common.mustCall());
+}).listen(common.PIPE, () => {
+  const client = net.connect(common.PIPE, common.mustCall());
+  client.on('data', () => {
+    client.end(() => {
+      server.close();
+    });
+  });
+});