Skip to content

fix(ext/node): count forked child process sockets#34096

Open
bartlomieju wants to merge 1 commit into
mainfrom
bartlomieju/fix-child-process-fork-getconnections
Open

fix(ext/node): count forked child process sockets#34096
bartlomieju wants to merge 1 commit into
mainfrom
bartlomieju/fix-child-process-fork-getconnections

Conversation

@bartlomieju
Copy link
Copy Markdown
Member

Summary

  • add fork IPC socket-list accounting for sockets transferred to child processes
  • include child-owned transferred sockets in net.Server#getConnections()
  • enable test-child-process-fork-getconnections.js in node compat config on non-Windows platforms

Tests

  • ./x build
  • ./x test-compat test-child-process-fork-getconnections.js
  • ./x test-compat test-child-process-send-keep-open.js test-child-process-fork-net-socket.js test-child-process-fork-net.js

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant