[java-idp-plugin-webauthn] branch main updated: Fix audit statement location

Phil Smart philip.smart at jisc.ac.uk
Wed Jul 31 15:29:26 UTC 2024


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=6b2d73959b673d1be1ff0313e165dc7d76493fca

The following commit(s) were added to refs/heads/main by this push:
     new 6b2d739  Fix audit statement location
6b2d739 is described below

commit 6b2d73959b673d1be1ff0313e165dc7d76493fca
Author: Phil Smart <philip.smart at jisc.ac.uk>
AuthorDate: Wed Jul 31 16:29:23 2024 +0100

    Fix audit statement location
    
     - Was only auditting if info logging was enabled
---
 .../plugin/authn/webauthn/admin/impl/StorePublicKeyCredential.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/webauthn-impl/src/main/java/net/shibboleth/idp/plugin/authn/webauthn/admin/impl/StorePublicKeyCredential.java b/webauthn-impl/src/main/java/net/shibboleth/idp/plugin/authn/webauthn/admin/impl/StorePublicKeyCredential.java
index a2a7ef1..9625516 100644
--- a/webauthn-impl/src/main/java/net/shibboleth/idp/plugin/authn/webauthn/admin/impl/StorePublicKeyCredential.java
+++ b/webauthn-impl/src/main/java/net/shibboleth/idp/plugin/authn/webauthn/admin/impl/StorePublicKeyCredential.java
@@ -158,9 +158,9 @@ public class StorePublicKeyCredential extends AbstractWebAuthnAuditingAction<Web
                         + "Using a discoverable credential '{}' and user verification '{}'", 
                         getLogPrefix(), username, userIdBase64, registrationResult.getKeyId().getId().getBase64Url(),
                         registrationResult.isDiscoverable().isPresent() ? registrationResult.isDiscoverable() : 
-                            "unknown", registrationResult.isUserVerified());
-                auditSuccess(profileRequestContext, "credential-added");
+                            "unknown", registrationResult.isUserVerified());                
             }
+            auditSuccess(profileRequestContext, "credential-added");
 
         } catch (final Exception e) {
             log.error("{} Unable to store registration for key '{}'",getLogPrefix(), 

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


More information about the commits mailing list