GH-55 Client emits connect event multiple times

lib/client.js

@@ -198,11 +198,9 @@ Client.prototype.connect = function(callback) {
           self.connection.end();
         }
 
-        self.emit('connect');
         return callback();
       });
 
-    self.emit('connect');
     return callback();
   });
 
@@ -854,7 +852,7 @@ Client.prototype._newConnection = function() {
 
       c.ldap.connected = true;
       c.ldap.id += c.fd ? (':' + c.fd) : '';
-      c.emit('connect', true);
+      c.emit('connect', c.ldap.id);
     });
     c.setKeepAlive = function(enable, delay) {
       return c.socket.setKeepAlive(enable, delay);

tst/laundry.test.js

@@ -52,6 +52,10 @@ test('setup', function(t) {
     t.end();
   });
 
+  server.bind('cn=root', function(req, res, next) {
+    res.end();
+    return next();
+  });
   server.search(SUFFIX, function(req, res, next) {
     var entry = {
       dn: 'cn=foo, ' + SUFFIX,
@@ -106,6 +110,26 @@ test('GH-49 Client errors on bad attributes', function(t) {
 });
 
 
+test('GH-55 Client emits connect multiple times', function(t) {
+  var c = ldap.createClient({
+    socketPath: SOCKET
+  });
+
+  var count = 0;
+  c.on('connect', function(socket) {
+    t.ok(socket);
+    count++;
+  });
+  c.bind('cn=root', 'secret', function(err, res) {
+    t.ifError(err);
+    c.unbind(function() {
+      t.equal(count, 1);
+      t.end();
+    });
+  });
+});
+
+
 test('shutdown', function(t) {
   client.unbind(function() {
     server.on('close', function() {