[java-idp-plugin-webauthn] branch main updated: Improve logging

[Phil Smart]

This is an automated email from the git hooks/post-receive script.

philsmart pushed a commit to branch main
in repository java-idp-plugin-webauthn.

View the commit online:
http://git.shibboleth.net/view/?p=java-idp-plugin-webauthn.git;a=commit;h=a2f7fcbb3bbb273d80c3991068a32fdabd730c1a

The following commit(s) were added to refs/heads/main by this push:
     new a2f7fcb  Improve logging
a2f7fcb is described below

commit a2f7fcbb3bbb273d80c3991068a32fdabd730c1a
Author: Phil Smart <philip.smart at jisc.ac.uk>
AuthorDate: Fri Sep 20 10:42:59 2024 +0100

    Improve logging
---
 .../webauthn/admin/impl/ValidateAuthenticatorAttestationResponse.java  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/webauthn-impl/src/main/java/net/shibboleth/idp/plugin/authn/webauthn/admin/impl/ValidateAuthenticatorAttestationResponse.java b/webauthn-impl/src/main/java/net/shibboleth/idp/plugin/authn/webauthn/admin/impl/ValidateAuthenticatorAttestationResponse.java
index 3a8c3aa..2668598 100644
--- a/webauthn-impl/src/main/java/net/shibboleth/idp/plugin/authn/webauthn/admin/impl/ValidateAuthenticatorAttestationResponse.java
+++ b/webauthn-impl/src/main/java/net/shibboleth/idp/plugin/authn/webauthn/admin/impl/ValidateAuthenticatorAttestationResponse.java
@@ -21,6 +21,7 @@ import org.opensaml.profile.action.ActionSupport;
 import org.opensaml.profile.context.ProfileRequestContext;
 import org.slf4j.Logger;
 
+import com.yubico.fido.metadata.AAGUID;
 import com.yubico.webauthn.data.AuthenticatorAttestationResponse;
 import com.yubico.webauthn.data.ByteArray;
 import com.yubico.webauthn.data.ClientRegistrationExtensionOutputs;
@@ -104,7 +105,7 @@ public class ValidateAuthenticatorAttestationResponse extends AbstractWebAuthnAc
             // untrusted attestations are allowed and the metadata has been loaded and the attestation statement has 
             // been validated.
             final ByteArray aaguid = credentialPublicKey.getAaguid();
-            final String athenticator = aaguid != null ? aaguid.getHex() : "unknown";
+            final String athenticator = aaguid != null ?new AAGUID(aaguid).asGuidString() : "unknown";
             log.debug("{} Was attestation for authenticator '{}' trusted? {}", getLogPrefix(), 
                     athenticator, credentialPublicKey.isAttestationTrusted() ? "Yes" : "No");
             // If valid. Add back to context

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.