fetchUsers: fix caching bug

Konfidens · Sep 30, 2023 · a11137e · a11137e
1 parent 5305e56
commit a11137e
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/data.ts b/src/data.ts
@@ -3,6 +3,7 @@ import fetch from "node-fetch";
 import { ChatBot, User } from "./types";
 
 let userData: User[] = [];
+let lastUpdated: number | undefined;
 
 async function isRedirected(url: string) {
   const data = await fetch(url, { method: "HEAD" });
@@ -28,12 +29,13 @@ const hasImageLazy = (user: User) => async () => {
 
 export async function fetchUsers({
   app,
-  refresh = false,
+  useCache = true,
 }: {
   app: ChatBot;
-  refresh?: false;
+  useCache?: true;
 }): Promise<User[]> {
-  if (userData && !refresh) {
+  const yesterday = new Date(Date.now() - 24 * 7 * 60 * 60 * 1000);
+  if (useCache && lastUpdated && lastUpdated > yesterday.getTime()) {
     return userData;
   }
   try {
@@ -55,7 +57,7 @@ export async function fetchUsers({
           hasImage: hasImageLazy(m as User),
         })
       );
-
+    lastUpdated = Date.now();
     return userData;
   } catch (error) {
     console.error(error);