From c4af2449c34215e87e43d3d25e33024de2fcb513 Mon Sep 17 00:00:00 2001
From: mini-bomba <55105495+mini-bomba@users.noreply.github.com>
Date: Sat, 1 Oct 2022 15:50:47 +0200
Subject: [PATCH] Use different startTime variables for each processTime check

This should make `processTime` and `redisProcessTime` values from /api/status more accurate
---
 src/routes/getStatus.ts | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/routes/getStatus.ts b/src/routes/getStatus.ts
index b817450..1f6dfcf 100644
--- a/src/routes/getStatus.ts
+++ b/src/routes/getStatus.ts
@@ -11,9 +11,10 @@ export async function getStatus(req: Request, res: Response): Promise<Response>
     value = Array.isArray(value) ? value[0] : value;
     let processTime, redisProcessTime = -1;
     try {
+        const dbStartTime = Date.now();
         const dbVersion = await promiseOrTimeout(db.prepare("get", "SELECT key, value FROM config where key = ?", ["version"]), 5000)
             .then(e => {
-                processTime = Date.now() - startTime;
+                processTime = Date.now() - dbStartTime;
                 return e.value;
             })
             .catch(e => {
@@ -21,9 +22,10 @@ export async function getStatus(req: Request, res: Response): Promise<Response>
                 return -1;
             });
         let statusRequests: unknown = 0;
+        const redisStartTime = Date.now();
         const numberRequests = await promiseOrTimeout(redis.increment("statusRequest"), 5000)
             .then(e => {
-                redisProcessTime = Date.now() - startTime;
+                redisProcessTime = Date.now() - redisStartTime;
                 return e;
             }).catch(e => {
                 Logger.error(`status: redis increment timed out ${e}`);