fix: ensure we use seconds everywhere

Author: simllll

storages/lru-redis/README.md

@@ -2,3 +2,5 @@
 
 LRU local cache and central redis cache,
 if not found local, redis is checked and in case there is a result it is saved locally
+
+all timeout values are in SECONDS

storages/lru-redis/src/LRUWithRedisStorage.ts

@@ -5,6 +5,7 @@ import * as Redis from "ioredis";
 
 export class LRUWithRedisStorage implements AsynchronousCacheType {
   private myCache: LRU<string, any>;
+  /** maxAge and ttl in seconds! */
   private options:  LRU.Options<string, any>;
 
   constructor(
@@ -16,7 +17,10 @@ export class LRUWithRedisStorage implements AsynchronousCacheType {
       maxAge: 86400,
       ...options
     }
-    this.myCache = new LRU(this.options);
+    this.myCache = new LRU({
+      ...this.options,
+      maxAge: (this.options.maxAge || 86400) * 1000 // in ms
+    });
   }
 
   public async getItem<T>(key: string): Promise<T | undefined> {
@@ -42,6 +46,7 @@ export class LRUWithRedisStorage implements AsynchronousCacheType {
     return localCache ?? undefined;
   }
 
+  /** ttl in seconds! */
   public async setItem(key: string, content: any, options?: { ttl?: number }): Promise<void> {
     this.myCache.set(key, content);
     if (this.options?.maxAge) {
@@ -53,6 +58,9 @@ export class LRUWithRedisStorage implements AsynchronousCacheType {
 
   public async clear(): Promise<void> {
     // flush not supported, recreate local lru cache instance
-    this.myCache = new LRU(this.options);
+    this.myCache = new LRU({
+      ...this.options,
+      maxAge: (this.options.maxAge || 86400) * 1000 // in ms
+    });
   }
 }

storages/lru/README.md

@@ -10,3 +10,5 @@ import LRUStorage from 'node-ts-cache-lru-storage';
 
 const myStrategy = new ExpirationStrategy(new LRUStorage());
 ```
+
+all timeout values are in SECONDS

storages/lru/src/LRUStorage.ts

@@ -9,8 +9,13 @@ export class LRUStorage
   implements SynchronousCacheType, MultiSynchronousCacheType {
   myCache: LRU<string, any>;
 
-  constructor(private options: LRU.Options<string, any>) {
-    this.myCache = new LRU(options);
+  constructor(
+    /** maxAge in seconds! */ private options: LRU.Options<string, any>
+  ) {
+    this.myCache = new LRU({
+      ...options,
+      maxAge: options.maxAge ? options.maxAge * 1000 : undefined,
+    });
   }
 
   getItems<T>(keys: string[]): { [key: string]: T | undefined } {
@@ -33,6 +38,9 @@ export class LRUStorage
 
   public clear(): void {
     // flush not supported, recreate lru cache instance
-    this.myCache = new LRU(this.options);
+    this.myCache = new LRU({
+      ...this.options,
+      maxAge: this.options.maxAge ? this.options.maxAge * 1000 : undefined, // in ms
+    });
   }
 }

storages/redisio/README.md

@@ -7,3 +7,5 @@ import RedisIOStorage from 'node-ts-cache-redisio-storage';
 
 const myStrategy = new ExpirationStrategy(new RedisIOStorage());
 ```
+
+all timeout values are in SECONDS