refactor(ELY-2597): merge if statements in verifyCertificate method

This merges an if statement with it's enclosing if statement to increase readability. This closes ticket https://issues.redhat.com/browse/ELY-2597
wildfly-security · Oct 6, 2023 · 42ca09c · 42ca09c
1 parent 5f8dd0f
commit 42ca09c
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/auth/realm/ldap/src/main/java/org/wildfly/security/auth/realm/ldap/X509EvidenceVerifier.java b/auth/realm/ldap/src/main/java/org/wildfly/security/auth/realm/ldap/X509EvidenceVerifier.java
@@ -176,10 +176,8 @@ public boolean verifyCertificate(X509Certificate certificate, Attributes attribu
 
                 for (int i = 0; i < size; i++) {
                     Object attrDigest = attribute.get(i);
-                    if (attrDigest != null){
-                        if (digest.equalsIgnoreCase((String) attrDigest)) {
-                            return true;
-                        }
+                    if (attrDigest != null && digest.equalsIgnoreCase((String) attrDigest)){
+                        return true;
                     }
                 }
             } catch (NoSuchAlgorithmException | CertificateEncodingException e) {