Remove the healthcheck from request/response logging

Refs #1834, d3b57c9
PREreview · Aug 28, 2024 · 37cea29 · 37cea29
1 parent 3fc0404
commit 37cea29
Showing 1 changed file with 21 additions and 21 deletions.
diff --git a/src/app.ts b/src/app.ts
@@ -135,6 +135,27 @@ const withEnv =
 export const app = (config: ConfigEnv) =>
   express()
     .disable('x-powered-by')
+    .get(
+      '/health',
+      asyncHandler(async (req, res) => {
+        if (config.redis === undefined) {
+          res.json({ status: 'ok' })
+          return
+        }
+        try {
+          if (config.redis.status !== 'ready') {
+            throw new Error(`Redis not ready (${config.redis.status})`)
+          }
+          await config.redis.ping()
+        } catch (error) {
+          const foo = toError(error)
+          L.errorP('healthcheck failed')({ message: foo.message, name: foo.name })(config)()
+          res.status(503).json({ status: 'error' })
+          return
+        }
+        res.json({ status: 'ok' })
+      }),
+    )
     .use((req, res, next) => {
       const url = new URL(req.url, config.publicUrl)
 
@@ -217,27 +238,6 @@ export const app = (config: ConfigEnv) =>
       res.type('text/plain')
       res.send('User-agent: *\nAllow: /')
     })
-    .get(
-      '/health',
-      asyncHandler(async (req, res) => {
-        if (config.redis === undefined) {
-          res.json({ status: 'ok' })
-          return
-        }
-        try {
-          if (config.redis.status !== 'ready') {
-            throw new Error(`Redis not ready (${config.redis.status})`)
-          }
-          await config.redis.ping()
-        } catch (error) {
-          const foo = toError(error)
-          L.errorP('healthcheck failed')({ message: foo.message, name: foo.name })(config)()
-          res.status(503).json({ status: 'error' })
-          return
-        }
-        res.json({ status: 'ok' })
-      }),
-    )
     .use(
       express.static('dist/assets', {
         setHeaders: (res, path) => {