tls: better error message for socket disconnect

The error emitted when a connection is closed before the
TLS handshake completes seemed rather unspefic by just saying
`socket hang up`.

Use a more verbose message, that also indicates that this is
a purely client-side error, and remove a misleading comment.

PR-URL: https://github.com/nodejs/node/pull/18989
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

1 files changed, 2 insertions, 2 deletions

diff --git a/lib/_tls_wrap.js b/lib/_tls_wrap.js
index b3c48b950c..c472564270 100644
--- a/lib/_tls_wrap.js
+++ b/lib/_tls_wrap.js
@@ -1074,7 +1074,6 @@ function onConnectSecure() {
     this.emit('secureConnect');
   }
 
-  // Uncork incoming data
   this.removeListener('end', onConnectEnd);
 }
 
@@ -1083,7 +1082,8 @@ function onConnectEnd() {
   if (!this._hadError) {
     const options = this[kConnectOptions];
     this._hadError = true;
-    const error = new Error('socket hang up');
+    const error = new Error('Client network socket disconnected before ' +
+                            'secure TLS connection was established');
     error.code = 'ECONNRESET';
     error.path = options.path;
     error.host = options.host;