fix: use lru cache replace node cache cooderl#141 cooderl#136

sanbens1234 · May 20, 2024 · 690b1c9 · 690b1c9
1 parent c487018
commit 690b1c9
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 23 deletions.
diff --git a/apps/server/package.json b/apps/server/package.json
@@ -43,7 +43,7 @@
     "got": "11.8.6",
     "hbs": "^4.2.0",
     "html-minifier": "^4.0.0",
-    "node-cache": "^5.1.2",
+    "lru-cache": "^10.2.2",
     "prisma": "^5.10.2",
     "reflect-metadata": "^0.2.1",
     "rxjs": "^7.8.1",

diff --git a/apps/server/src/feeds/feeds.service.ts b/apps/server/src/feeds/feeds.service.ts
@@ -10,13 +10,13 @@ import { Feed } from 'feed';
 import got, { Got } from 'got';
 import { load } from 'cheerio';
 import { minify } from 'html-minifier';
-import NodeCache from 'node-cache';
+import { LRUCache } from 'lru-cache';
 import pMap from '@cjs-exporter/p-map';
 
 console.log('CRON_EXPRESSION: ', process.env.CRON_EXPRESSION);
 
-const mpCache = new NodeCache({
-  maxKeys: 1000,
+const mpCache = new LRUCache<string, string>({
+  max: 5000,
 });
 
 @Injectable()
@@ -120,7 +120,7 @@ export class FeedsService {
   }
 
   async tryGetContent(id: string) {
-    let content = mpCache.get(id) as string;
+    let content = mpCache.get(id);
     if (content) {
       return content;
     }

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml