process disconnect packet w/o full destroy the connection (#937)

* process disconnect packet w/o full destroy the connection * Update README.md Co-Authored-By: Matteo Collina <matteo.collina@gmail.com>
mqttjs · May 14, 2019 · 84ca344 · 84ca344
1 parent e8de240
commit 84ca344
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -291,6 +291,12 @@ Emitted when a reconnect starts.
 
 Emitted after a disconnection.
 
+#### Event `'disconnect'`
+
+`function (packet) {}`
+
+Emitted after receiving disconnect packet from broker. MQTT 5.0 feature.
+
 #### Event `'offline'`
 
 `function () {}`

diff --git a/lib/client.js b/lib/client.js
@@ -1277,8 +1277,7 @@ MqttClient.prototype._handlePubrel = function (packet, callback) {
  * @api private
  */
 MqttClient.prototype._handleDisconnect = function (packet) {
-  this.emit('close', packet)
-  this.end(true)
+  this.emit('disconnect', packet)
 }
 
 /**

diff --git a/test/client.js b/test/client.js
@@ -953,7 +953,7 @@ describe('MqttClient', function () {
       }
 
       var client = mqtt.connect(opts)
-      client.once('close', function (disconnectPacket) {
+      client.once('disconnect', function (disconnectPacket) {
         should(disconnectPacket.reasonCode).be.equal(128)
         done()
       })