fix(multicache): handle get key undefined

Author: simllll

storages/lru-redis/package.json

@@ -54,5 +54,5 @@
     "@types/lru-cache": "^5.1.0",
     "ioredis-mock": "^4.21.8"
   },
-  "gitHead": "12356d80b38a7b841fa377840f13c37fbe3c9cd3"
+  "gitHead": "c938eba762060f940a34bc192bec03bc76ea4017"
 }

storages/lru/package.json

@@ -51,5 +51,5 @@
   "devDependencies": {
     "@types/lru-cache": "^5.1.0"
   },
-  "gitHead": "179ca6def52cbf51e09fc22c09f7313d6356105b"
+  "gitHead": "c938eba762060f940a34bc192bec03bc76ea4017"
 }

storages/node-cache/package.json

@@ -51,5 +51,5 @@
     "@hokify/node-ts-cache": "^5.4.0",
     "@types/node-cache": "^4.2.5"
   },
-  "gitHead": "179ca6def52cbf51e09fc22c09f7313d6356105b"
+  "gitHead": "c938eba762060f940a34bc192bec03bc76ea4017"
 }

storages/redis/package.json

@@ -63,5 +63,5 @@
     "proxyquire": "2.1.3",
     "sinon": "9.2.0"
   },
-  "gitHead": "179ca6def52cbf51e09fc22c09f7313d6356105b"
+  "gitHead": "c938eba762060f940a34bc192bec03bc76ea4017"
 }

storages/redisio/package.json

@@ -54,5 +54,5 @@
     "@types/ioredis": "^4.17.6",
     "ioredis-mock": "^4.21.8"
   },
-  "gitHead": "7d2e5b85914c7a6e54edad434b0fcbae37c484b4"
+  "gitHead": "c938eba762060f940a34bc192bec03bc76ea4017"
 }

ts-cache/package.json

@@ -53,5 +53,5 @@
     "url": "https://github.com/hokify/node-ts-cache/issues"
   },
   "homepage": "https://github.com/hokify/node-ts-cache/tree/master/ts-cache#readme",
-  "gitHead": "179ca6def52cbf51e09fc22c09f7313d6356105b"
+  "gitHead": "c938eba762060f940a34bc192bec03bc76ea4017"
 }

ts-cache/src/decorator/multicache.decorator.ts

@@ -7,7 +7,7 @@ const defaultKeyStrategy = {
     parameter: any,
     args: any,
     _phase: 'read' | 'write'
-  ): string {
+  ): string | undefined {
     return `${className}:${methodName}:${JSON.stringify(
       parameter
     )}:${JSON.stringify(args)}`;
@@ -136,17 +136,22 @@ export function MultiCache(
         if (!process.env.DISABLE_CACHE_DECORATOR) {
           // save back to all caching strategies
           const saveToCache = originalMethodResult.map((content, i) => {
-            return {
-              key: keyStrategy.getKey(
+            const key = keyStrategy.getKey(
                 className,
                 methodName,
                 missingKeys[i],
                 args,
                 'write'
-              ),
+            );
+            if (key === undefined) {
+              return undefined;
+            }
+
+            return {
+              key,
               content,
             };
-          });
+          }).filter(f => f !== undefined) as {key: string; content: any}[]
 
           // console.log('saveToCache', saveToCache);