Skip heartbeat on /healthz endpoint

I managed to lose this in the rewrite. Fixes #2327.
coder · Nov 18, 2020 · 51c3709 · 51c3709
1 parent 5e60305
commit 51c3709
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/node/routes/index.ts b/src/node/routes/index.ts
@@ -66,7 +66,11 @@ export const register = async (
   app.use(bodyParser.urlencoded({ extended: true }))
 
   const common: express.RequestHandler = (req, _, next) => {
-    heart.beat()
+    // /healthz|/healthz/ needs to be excluded otherwise health checks will make
+    // it look like code-server is always in use.
+    if (!/^\/healthz\/?$/.test(req.url)) {
+      heart.beat()
+    }
 
     // Add common variables routes can use.
     req.args = args