removed mutiple hand shake error events

Author: sushantdhiman

lib/commands/client_handshake.js

@@ -68,10 +68,10 @@ ClientHandshake.prototype.calculateNativePasswordAuthToken = function (authPlugi
 ClientHandshake.prototype.handshakeInit = function (helloPacket, connection) {
   var command = this;
 
-  this.on('error', function (err) {
-    connection._protocolError = err;
-    connection.emit('error', err);
+  this.on('error', function (e) {
+    connection._protocolError = e;
   });
+
   this.handshake = Packets.Handshake.fromPacket(helloPacket);
   if (connection.config.debug) {
     console.log('Server hello packet: capability flags:%d=(%s)', this.handshake.capabilityFlags,

lib/connection.js

@@ -103,19 +103,17 @@ function Connection (opts)
       return;
     }
 
-    // TODO: move to protocolError()
     if (!connection._protocolError) { // no particular error message before disconnect
-      connection._protocolError = 'PROTOCOL_CONNECTION_LOST';
+      connection._protocolError = new Error('Connection lost: The server closed the connection.');
+      connection._protocolError.fatal = true;
+      connection._protocolError.code = 'PROTOCOL_CONNECTION_LOST';
     }
-    var err = new Error('Connection lost: The server closed the connection.');
-    err.fatal = true;
-    err.code = connection._protocolError;
-    connection._notifyError(err);
+
+    connection._notifyError(connection._protocolError);
   });
   var handshakeCommand;
   if (!this.config.isServer) {
     handshakeCommand = new Commands.ClientHandshake(this.config.clientFlags);
-    handshakeCommand.on('error', function (e) { connection.emit('error', e); });
     handshakeCommand.on('end', function () {
       connection._handshakePacket = handshakeCommand.handshake;
       connection.threadId = handshakeCommand.handshake.connectionId;