fix: /health return 200 if mqtt is disabled (#500)

Fixes #469
zwave-js · Feb 8, 2021 · 8e3a4e3 · 8e3a4e3
1 parent 6a9931f
commit 8e3a4e3
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/app.js b/app.js
@@ -213,10 +213,15 @@ app.get('/health', async function (req, res) {
   let zwave = false
 
   if (gw) {
-    mqtt = gw.mqtt ? gw.mqtt.getStatus().status : false
+    mqtt = gw.mqtt ? gw.mqtt.getStatus() : false
     zwave = gw.zwave ? gw.zwave.getStatus().status : false
   }
 
+  // if mqtt is disabled, return true. Fixes #469
+  if (mqtt) {
+    mqtt = mqtt.status || mqtt.config.disabled
+  }
+
   const status = mqtt && zwave
 
   res.status(status ? 200 : 500).send(status ? 'Ok' : 'Error')