Skip to content

Commit cc5de5a

Browse files
lpincaaddaleax
authored andcommitted
http: remove redundant call to socket.setTimeout()
`Socket.prototype.setTimeout()` clears the previous timer before setting a new one. Refs: #25748 (comment) PR-URL: #25928 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 00d49ad commit cc5de5a

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

lib/_http_server.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -598,7 +598,6 @@ function resOnFinish(req, res, socket, state, server) {
598598
}
599599
} else if (state.outgoing.length === 0) {
600600
if (server.keepAliveTimeout && typeof socket.setTimeout === 'function') {
601-
socket.setTimeout(0);
602601
socket.setTimeout(server.keepAliveTimeout);
603602
state.keepAliveTimeoutSet = true;
604603
}

0 commit comments

Comments
 (0)