fix: did cache key not being set correctly (#1394)

Signed-off-by: Timo Glastra <timo@animo.id>
openwallet-foundation · Mar 21, 2023 · 1125e81 · 1125e81
1 parent dbfebb4
commit 1125e81
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/packages/indy-sdk/src/ledger/IndySdkPoolService.ts b/packages/indy-sdk/src/ledger/IndySdkPoolService.ts
@@ -98,7 +98,9 @@ export class IndySdkPoolService {
     }
 
     const cache = agentContext.dependencyManager.resolve(CacheModuleConfig).cache
-    const cachedNymResponse = await cache.get<CachedDidResponse>(agentContext, `IndySdkPoolService:${did}`)
+    const cacheKey = `IndySdkPoolService:${did}`
+
+    const cachedNymResponse = await cache.get<CachedDidResponse>(agentContext, cacheKey)
     const pool = this.pools.find((pool) => pool.didIndyNamespace === cachedNymResponse?.indyNamespace)
 
     // If we have the nym response with associated pool in the cache, we'll use that
@@ -145,7 +147,7 @@ export class IndySdkPoolService {
       value = productionOrNonProduction[0].value
     }
 
-    await cache.set(agentContext, `IndySdkPoolService:${did}`, {
+    await cache.set(agentContext, cacheKey, {
       nymResponse: value.did,
       indyNamespace: value.pool.didIndyNamespace,
     } satisfies CachedDidResponse)

diff --git a/packages/indy-vdr/src/pool/IndyVdrPoolService.ts b/packages/indy-vdr/src/pool/IndyVdrPoolService.ts
@@ -73,9 +73,10 @@ export class IndyVdrPoolService {
       )
     }
 
-    const didCache = agentContext.dependencyManager.resolve(CacheModuleConfig).cache
+    const cache = agentContext.dependencyManager.resolve(CacheModuleConfig).cache
+    const cacheKey = `IndyVdrPoolService:${did}`
 
-    const cachedNymResponse = await didCache.get<CachedDidResponse>(agentContext, `IndyVdrPoolService:${did}`)
+    const cachedNymResponse = await cache.get<CachedDidResponse>(agentContext, cacheKey)
     const pool = this.pools.find((pool) => pool.indyNamespace === cachedNymResponse?.indyNamespace)
 
     // If we have the nym response with associated pool in the cache, we'll use that
@@ -122,7 +123,7 @@ export class IndyVdrPoolService {
       value = productionOrNonProduction[0].value
     }
 
-    await didCache.set(agentContext, did, {
+    await cache.set(agentContext, cacheKey, {
       nymResponse: {
         did: value.did.nymResponse.did,
         verkey: value.did.nymResponse.verkey,