Generic error event for missing client Ids

README.md

@@ -115,6 +115,9 @@ Events:
 * `clientError`: when a [Client](#client) errors, arguments:
   1. `client`
   2. `err`
+* `connectionError` When a [Client](#client) connection errors and there is no clientId attached , arguments:
+  1. `client`
+  2. `err`
 * `keepaliveTimeout`: when a [Client](#client) keepalive times out, arguments:
   1. `client`
 * `publish`: when a new packet is published, arguments:

example.js

@@ -23,6 +23,10 @@ aedes.on('clientError', function (client, err) {
   console.log('client error', client.id, err.message, err.stack)
 })
 
+aedes.on('connectionError', function (client, err) {
+  console.log('client error', client, err.message, err.stack)
+})
+
 aedes.on('publish', function (packet, client) {
   if (client) {
     console.log('message from client', client.id)

lib/client.js

@@ -151,7 +151,11 @@ function onError (err) {
   this.errored = true
   this.conn.removeAllListeners('error')
   this.conn.on('error', nop)
-  this.broker.emit('clientError', this, err)
+  if (this.id) {
+    this.broker.emit('clientError', this, err)
+  } else {
+    this.broker.emit('connectionError', this, err)
+  }
   this.close()
 }
 

test/basic.js

@@ -321,6 +321,17 @@ test('closes', function (t) {
   })
 })
 
+test('testing other event', function (t) {
+  var broker = aedes()
+  var client = setup(broker)
+
+  broker.on('connectionError', function (client, error) {
+    t.notOk(client.id, null)
+    t.end()
+  })
+  client.conn.emit('error', 'Connect not yet arrived')
+})
+
 test('connect without a clientId for MQTT 3.1.1', function (t) {
   var s = setup()