Skip to content

http: replace destroySoon with socket.end

Rodrigo Muino Tomonari requested to merge github/fork/mmomtchev/halfclose-rst into main

Closing a TCP connection requires 2 FIN/ACK exchanges, so we should wait for both events before destroying a socket

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

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

Merge request reports