child_process: fix channel disconnect logic & cleanup pending handles (!19439) · Merge requests · Rodrigo Test / Test Group-nodejs / node · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Rodrigo Muino Tomonari requested to merge github/fork/laino/child-process-close into master Mar 19, 2018

Fix close not being emitted when calling process.disconnect() from a parent process.
When there are pending handles that would never be closed because the other side has closed the channel, they are now closed 'manually'.
Extend child disconnect test case to also check that 'close' was emitted.

Fixes: https://github.com/nodejs/node/issues/19433

Checklist

make -j4 test (UNIX), or vcbuild test (Windows) passes
tests and/or benchmarks are included
commit message follows commit guidelines