Skip heartbeat on /healthz endpoint (#2333)

code-asher · web-flow · commit 2a3608df533f · 2020-11-18T12:19:08.000-06:00
I managed to lose this in the rewrite. Fixes #2327.
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
