simplified getSupportedPinUvAuthProtocol

Yubico · Oct 31, 2023 · 2666e4e · 2666e4e
1 parent ec02b1f
commit 2666e4e
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/fido/src/main/java/com/yubico/yubikit/fido/client/BasicWebAuthnClient.java b/fido/src/main/java/com/yubico/yubikit/fido/client/BasicWebAuthnClient.java
@@ -609,13 +609,15 @@ private boolean getCtapUv(String userVerification, boolean pinProvided) throws C
      * supported protocols.
      */
     private PinUvAuthProtocol getPreferredPinUvAuthProtocol(List<Integer> pinUvAuthProtocols) {
-        for (int protocol : pinUvAuthProtocols) {
-            if (pinSupported && protocol == PinUvAuthProtocolV1.VERSION) {
-                return new PinUvAuthProtocolV1();
-            }
+        if (pinSupported) {
+            for (int protocol : pinUvAuthProtocols) {
+                if (protocol == PinUvAuthProtocolV1.VERSION) {
+                    return new PinUvAuthProtocolV1();
+                }
 
-            if (pinSupported && protocol == PinUvAuthProtocolV2.VERSION) {
-                return new PinUvAuthProtocolV2();
+                if (protocol == PinUvAuthProtocolV2.VERSION) {
+                    return new PinUvAuthProtocolV2();
+                }
             }
         }