Merge pull request #869 from azisaka/feat/redis-cache-default-ttl

feat: define a default TTL specifically for redis cache
deco-cx · Nov 19, 2024 · ebe6471 · ebe6471
2 parents 15e0d76 + a54665c
commit ebe6471
Showing 1 changed file with 2 additions and 9 deletions.
diff --git a/runtime/caches/redis.ts b/runtime/caches/redis.ts
@@ -15,6 +15,7 @@ import {
 
 const CONNECTION_TIMEOUT = 500;
 const RECONNECTION_TIMEOUT = 5000;
+const TTL = parseInt(Deno.env.get("LOADER_CACHE_REDIS_TTL_SECONDS") || "3600");
 
 type RedisConnection = RedisClientType<
   RedisModules,
@@ -119,15 +120,7 @@ export const caches: CacheStorage = {
         const cacheKey = await generateKey(request);
 
         serialize(response)
-          .then((data) => {
-            const expirationTimestamp = Date.parse(
-              response.headers.get("expires") ?? "",
-            );
-
-            const ttl = expirationTimestamp - Date.now();
-
-            return redis?.set(cacheKey, data, { PX: ttl });
-          })
+          .then((data) => redis?.set(cacheKey, data, { EX: TTL }))
           .catch(() => {});
       },
     });