aedes 0.51.1 → 0.51.2

package/lib/client.js

@@ -313,7 +313,14 @@ Client.prototype.close = function (done) {
           }, noop)
         }
       })
+    } else if (will) {
+      // delete the persisted will even on clean disconnect https://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.html#_Toc385349232
+      that.broker.persistence.delWill({
+        id: that.id,
+        brokerId: that.broker.id
+      }, noop)
     }
+
     that.will = null // this function might be called twice
     that._will = null
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aedes",
-  "version": "0.51.1",
+  "version": "0.51.2",
   "description": "Stream-based MQTT broker",
   "main": "aedes.js",
   "types": "aedes.d.ts",
@@ -108,7 +108,7 @@
     "concat-stream": "^2.0.0",
     "duplexify": "^4.1.2",
     "license-checker": "^25.0.1",
-    "markdownlint-cli": "^0.39.0",
+    "markdownlint-cli": "^0.41.0",
     "mqtt": "^5.3.5",
     "mqtt-connection": "^4.1.0",
     "pre-commit": "^1.2.2",
@@ -117,7 +117,7 @@
     "snazzy": "^9.0.0",
     "standard": "^17.1.0",
     "tap": "^16.3.10",
-    "tsd": "^0.30.4",
+    "tsd": "^0.31.0",
     "typescript": "^5.3.3",
     "websocket-stream": "^5.5.2"
   },
@@ -129,7 +129,7 @@
     "fastparallel": "^2.4.1",
     "fastseries": "^2.0.0",
     "hyperid": "^3.2.0",
-    "mqemitter": "^5.0.0",
+    "mqemitter": "^6.0.0",
     "mqtt-packet": "^9.0.0",
     "retimer": "^4.0.0",
     "reusify": "^1.0.4",

package/test/will.js

@@ -420,6 +420,29 @@ test('does not deliver will when client sends a DISCONNECT', function (t) {
   })
 })
 
+test('deletes from persistence on DISCONNECT', function (t) {
+  t.plan(2)
+
+  const opts = {
+    clientId: 'abcde'
+  }
+  const broker = aedes()
+  t.teardown(broker.close.bind(broker))
+
+  const s = noError(willConnect(setup(broker), opts, function () {
+    s.inStream.end({
+      cmd: 'disconnect'
+    })
+  }), t)
+
+  s.broker.persistence.getWill({
+    id: opts.clientId
+  }, function (err, packet) {
+    t.error(err, 'no error')
+    t.notOk(packet)
+  })
+})
+
 test('does not store multiple will with same clientid', function (t) {
   t.plan(4)