apply review suggestions

Author: anonrig

src/node/internal/internal_tls_wrap.ts

@@ -320,7 +320,7 @@ TLSSocket.prototype._init = function _init(this: TLSSocket): void {
 
   this.on('error', onerror.bind(this));
 
-  // This is thrown from net.Socket class.
+  // This is emitted from net.Socket class.
   this.on('connectionAttempt', onnewsessionclient.bind(this));
 
   if (options.handshakeTimeout && options.handshakeTimeout > 0)
@@ -418,21 +418,15 @@ TLSSocket.prototype._finishInit = function _finishInit(this: TLSSocket): void {
   if (
     this._tlsOptions.handshakeTimeout &&
     this._tlsOptions.handshakeTimeout > 0
-  )
+  ) {
     this.setTimeout(0, this._handleTimeout.bind(this));
+  }
+
   this.emit('secure');
 };
 
 TLSSocket.prototype._start = function _start(this: TLSSocket): void {
-  if (this.connecting) {
-    this.once('connect', this._start.bind(this));
-    return;
-  }
-
-  // Socket was destroyed before the connection was established
-  if (!this._handle) return;
-
-  // this._handle.start();
+  // Do nothing.
 };
 
 TLSSocket.prototype.setServername = function setServername(

src/workerd/api/node/tests/tls-nodejs-tcp-server.js

@@ -2,9 +2,7 @@
 // Licensed under the Apache 2.0 license found in the LICENSE file or at:
 //     https://opensource.org/licenses/Apache-2.0
 
-// This file is used as a sidecar for the net-nodejs-test tests.
-// It creates 2 TCP servers to act as a source of truth for the node:net tests.
-// We execute this command using Node.js, which makes net.createServer available.
+// This file is used as a sidecar for the tls-nodejs-test tests.
 const tls = require('node:tls');
 
 // Taken from https://github.com/nodejs/node/blob/304743655d5236c2edc39094336ee2667600b684/test/fixtures/keys/agent1-key.pem