Fix args to CredentialManagement

Yubico · Oct 9, 2024 · 99e0491 · 99e0491
1 parent 0c50d92
commit 99e0491
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/fido2/ctap2/credman.py b/fido2/ctap2/credman.py
@@ -217,7 +217,8 @@ def delete_cred(self, cred_id: PublicKeyCredentialDescriptor) -> None:
 
         :param cred_id: The PublicKeyCredentialDescriptor of the credential to delete.
         """
-        logger.debug(f"Deleting credential with ID: {cred_id['id'].hex()}")
+        cred_id = PublicKeyCredentialDescriptor.from_dict(cred_id)
+        logger.debug(f"Deleting credential with ID: {cred_id}")
         self._call(
             CredentialManagement.CMD.DELETE_CREDENTIAL,
             {CredentialManagement.PARAM.CREDENTIAL_ID: cred_id},
@@ -237,6 +238,7 @@ def update_user_info(
         if not CredentialManagement.is_update_supported(self.ctap.info):
             raise ValueError("Authenticator does not support update_user_info")
 
+        cred_id = PublicKeyCredentialDescriptor.from_dict(cred_id)
         logger.debug(f"Updating credential: {cred_id} with user info: {user_info}")
         self._call(
             CredentialManagement.CMD.UPDATE_USER_INFO,