From 9c60b4b93da37a897a9a6399f0ba8155f25d5ef3 Mon Sep 17 00:00:00 2001
From: Yanlong Wang <yanlong.wang@naiver.org>
Date: Fri, 31 May 2024 18:36:23 +0800
Subject: [PATCH] fix: setup expire for pdf caches

---
 backend/functions/src/services/pdf-extract.ts | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/backend/functions/src/services/pdf-extract.ts b/backend/functions/src/services/pdf-extract.ts
index ee9296a..ccb7c52 100644
--- a/backend/functions/src/services/pdf-extract.ts
+++ b/backend/functions/src/services/pdf-extract.ts
@@ -46,6 +46,8 @@ export class PDFExtractor extends AsyncService {
     logger = this.globalLogger.child({ service: this.constructor.name });
     pdfjs!: Awaited<typeof pPdfjs>;
 
+    cacheRetentionMs = 1000 * 3600 * 24 * 7;
+
     constructor(
         protected globalLogger: Logger,
         protected firebaseObjectStorage: FirebaseStorageBucketControl,
@@ -278,8 +280,12 @@ export class PDFExtractor extends AsyncService {
             this.logger.warn(`Unable to extract from pdf ${url}`, { err });
         }
 
-        // Don't try again until the next day
-        const expireMixin = extracted ? {} : { expireAt: new Date(Date.now() + 1000 * 3600 * 24) };
+        // Don't try again until the next hour
+        const expireMixin = extracted ? {
+            expireAt: new Date(Date.now() + this.cacheRetentionMs)
+        } : {
+            expireAt: new Date(Date.now() + 1000 * 3600)
+        };
         const theID = randomUUID();
 
         await this.firebaseObjectStorage.saveFile(`pdfs/${theID}`,