fix louislam#3767 by @nougad - treat empty healthcheck reported by po…

…dman as up
vfosnar · Jan 19, 2024 · 7f5dfcc · 7f5dfcc
1 parent d736841
commit 7f5dfcc
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/server/model/monitor.js b/server/model/monitor.js
@@ -770,12 +770,12 @@ class Monitor extends BeanModel {
                     let res = await axios.request(options);
 
                     if (res.data.State.Running) {
-                        if (res.data.State.Health && res.data.State.Health.Status === "unhealthy") {
-                            bean.status = DOWN;
-                            bean.msg = res.data.State.Health.Status;
-                        } else if (res.data.State.Health && res.data.State.Health.Status !== "healthy") {
+                        if (res.data.State.Health && ![ "healthy", "unhealthy", "" ].includes(res.data.State.Health.Status)) {
                             bean.status = PENDING;
                             bean.msg = res.data.State.Health.Status;
+                        } else if (res.data.State.Health && res.data.State.Health.Status === "unhealthy") {
+                            bean.status = DOWN;
+                            bean.msg = res.data.State.Health.Status;
                         } else {
                             bean.status = UP;
                             bean.msg = res.data.State.Health ? res.data.State.Health.Status : res.data.State.Status;