tls: remove unnecessary close listener

ronag · ronag · commit 60a217b1ea8c · 2020-07-01T18:18:59.000+02:00
Wrapped streams are expected to behave the same as socket with handle. Remove unnecessary difference in handling. PR-URL: nodejs#34105 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
diff --git a/lib/_tls_wrap.js b/lib/_tls_wrap.js
@@ -490,7 +490,6 @@ function TLSSocket(socket, opts) {
     // handle, but a JS stream doesn't have one. Wrap it up to make it look like
     // a socket.
     wrap = new JSStreamSocket(socket);
-    wrap.once('close', () => this.destroy());
   }
 
   // Just a documented property to make secure sockets

Original file line number	Diff line number	Diff line change
`@@ -490,7 +490,6 @@ function TLSSocket(socket, opts) {`
`490`	`490`	`// handle, but a JS stream doesn't have one. Wrap it up to make it look like`
`491`	`491`	`// a socket.`
`492`	`492`	`wrap = new JSStreamSocket(socket);`
`493`		`- wrap.once('close', () => this.destroy());`
`494`	`493`	`}`
`495`	`494`
`496`	`495`	`// Just a documented property to make secure sockets`