code review changes

packages/health-check/src/api/critical.js

@@ -1,8 +1,14 @@
 const db = require("../db");
+const { getYesterdayISOString } = require("../utils");
 
 // returns all critical health check entries
 module.exports = (req, res) => {
-  const entries = db.get("critical").orderBy("date", "desc").value();
+  const yesterday = getYesterdayISOString();
+  const entries = db
+    .get("critical")
+    .orderBy("date", "desc")
+    .filter(({ date }) => date > yesterday)
+    .value();
 
   res.send(entries);
 };

packages/health-check/src/api/index.js

@@ -18,13 +18,12 @@ function getStatusCode() {
   // grab the most recent critical entry element from DB
   const entry = getCurrentCriticalEntry();
 
-  // find out whether every check in the entry is up
+  // in case there is no entry yet or at least one check failed in the most recent entry
-  if (entry && entry.checks.every(({ up }) => up)) {
+  if (!entry || entry.checks.some(({ up }) => !up)) {
-    return StatusCodes.OK;
+    return StatusCodes.SERVICE_UNAVAILABLE;
   }
 
-  // in case at least one check failed
+  return StatusCodes.OK;
-  return StatusCodes.SERVICE_UNAVAILABLE;
 }
 
 /**

packages/health-check/src/api/verbose.js

@@ -1,8 +1,14 @@
 const db = require("../db");
+const { getYesterdayISOString } = require("../utils");
 
 // returns all verbose health check entries
 module.exports = (req, res) => {
-  const entries = db.get("verbose").orderBy("date", "desc").value();
+  const yesterday = getYesterdayISOString();
+  const entries = db
+    .get("verbose")
+    .orderBy("date", "desc")
+    .filter(({ date }) => date > yesterday)
+    .value();
 
   res.send(entries);
 };

packages/health-check/src/utils.js

@@ -1,8 +1,21 @@
-// return the time between start and now in milliseconds
+/**
+ * Get the time between start and now in milliseconds
+ */
 function calculateElapsedTime(start) {
   const diff = process.hrtime(start);
 
   return Math.round((diff[0] * 1e9 + diff[1]) / 1e6); // msec
 }
 
-module.exports = { calculateElapsedTime };
+/**
+ * Get the ISO string with yesterday's date set (- 24 hours)
+ */
+function getYesterdayISOString() {
+  const date = new Date();
+
+  date.setDate(date.getDate() - 1);
+
+  return date.toISOString();
+}
+
+module.exports = { calculateElapsedTime, getYesterdayISOString };