[BUG] Handle get key operation when no tag has been set (openwallet-f…

…oundation#3256) * Bug-fix Signed-off-by: Patrick St-Louis <[email protected]> * Update in_memory.py Signed-off-by: Patrick St-Louis <[email protected]> * linting Signed-off-by: PatStLouis <[email protected]> * apply better suggestion Signed-off-by: PatStLouis <[email protected]> * set empty tags object instead of none for consistency Signed-off-by: PatStLouis <[email protected]> * remove unnecessary change in the in memory wallet Signed-off-by: PatStLouis <[email protected]> --------- Signed-off-by: Patrick St-Louis <[email protected]> Signed-off-by: PatStLouis <[email protected]>
jamshale · Sep 26, 2024 · 9163871 · 9163871
1 parent dc66c82
commit 9163871
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/aries_cloudagent/wallet/askar.py b/aries_cloudagent/wallet/askar.py
@@ -97,7 +97,7 @@ async def create_key(
         if kid:
             tags = {"kid": kid}
         else:
-            tags = None
+            tags = {}
 
         try:
             keypair = _create_keypair(key_type, seed)
@@ -192,8 +192,10 @@ async def get_signing_key(self, verkey: str) -> KeyInfo:
         if not key:
             raise WalletNotFoundError("Unknown key: {}".format(verkey))
         metadata = json.loads(key.metadata or "{}")
+
+        kid = key.tags.get("kid")
+
         # FIXME implement key types
-        kid = key.tags["kid"] if "kid" in key.tags else None
         return KeyInfo(verkey=verkey, metadata=metadata, key_type=ED25519, kid=kid)
 
     async def replace_signing_key_metadata(self, verkey: str, metadata: dict):