From aa3a78c744bdecbc49cfb38ef2bfd5bec9bd8fb8 Mon Sep 17 00:00:00 2001
From: Nicolai Schmid <nicolai@schmid.uno>
Date: Sun, 1 May 2022 22:03:56 +0200
Subject: [PATCH] feat(health): implement proper health check by pinging redis

---
 lib/api-server.js | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/lib/api-server.js b/lib/api-server.js
index aa9809dd..ac518ab0 100644
--- a/lib/api-server.js
+++ b/lib/api-server.js
@@ -76,12 +76,17 @@ class APIServer {
     setupRoutes() {
         this.server.get('/healthz', (req, res, next) => {
             const redis = plugins.handler.options.db.redis;
-            console.log(redis);
 
-            res.json(200, {
-                redis: true
+            redis.ping((err, result) => {
+                console.log(err, result);
+                const status = err ? 500 : 200;
+
+                res.json(status, {
+                    redis: err ? err : result
+                });
+
+                return next();
             });
-            return next();
         });
 
         // Example handler for authentication. You should probably override it