diff options
author | Bjørn Christian Seime <bjorncs@yahooinc.com> | 2022-07-13 17:22:04 +0200 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@yahooinc.com> | 2022-07-15 15:35:10 +0200 |
commit | 64adc479fab2fad65c398e70222f3443b75f9f32 (patch) | |
tree | 614120bdf92ede7f4a26cd449bf27f1591f45843 /security-utils | |
parent | e69c68a2c4b9b8f8d556f376c8f023f602a95eff (diff) |
Rename 'succeeded' => 'authorized'
Diffstat (limited to 'security-utils')
3 files changed, 4 insertions, 4 deletions
diff --git a/security-utils/src/main/java/com/yahoo/security/tls/authz/ConnectionAuthContext.java b/security-utils/src/main/java/com/yahoo/security/tls/authz/ConnectionAuthContext.java index 18f61fc7aa4..52d838d29ef 100644 --- a/security-utils/src/main/java/com/yahoo/security/tls/authz/ConnectionAuthContext.java +++ b/security-utils/src/main/java/com/yahoo/security/tls/authz/ConnectionAuthContext.java @@ -21,7 +21,7 @@ public record ConnectionAuthContext(List<X509Certificate> peerCertificateChain, matchedPolicies = new TreeSet<>(matchedPolicies); } - public boolean succeeded() { return matchedPolicies.size() > 0; } + public boolean authorized() { return matchedPolicies.size() > 0; } public X509Certificate peerCertificate() { return peerCertificateChain.get(0); } diff --git a/security-utils/src/main/java/com/yahoo/security/tls/authz/PeerAuthorizerTrustManager.java b/security-utils/src/main/java/com/yahoo/security/tls/authz/PeerAuthorizerTrustManager.java index ea920a90c7b..21a089e4295 100644 --- a/security-utils/src/main/java/com/yahoo/security/tls/authz/PeerAuthorizerTrustManager.java +++ b/security-utils/src/main/java/com/yahoo/security/tls/authz/PeerAuthorizerTrustManager.java @@ -113,7 +113,7 @@ public class PeerAuthorizerTrustManager extends X509ExtendedTrustManager { if (sslEngine != null) { // getHandshakeSession() will never return null in this context sslEngine.getHandshakeSession().putValue(HANDSHAKE_SESSION_AUTH_CONTEXT_PROPERTY, result); } - if (result.succeeded()) { + if (result.authorized()) { log.fine(() -> String.format("Verification result: %s", result)); } else { String errorMessage = "Authorization failed: " + createInfoString(certChain[0], authType, isVerifyingClient); diff --git a/security-utils/src/test/java/com/yahoo/security/tls/authz/PeerAuthorizerTest.java b/security-utils/src/test/java/com/yahoo/security/tls/authz/PeerAuthorizerTest.java index a2f27ba42bc..3791aed4155 100644 --- a/security-utils/src/test/java/com/yahoo/security/tls/authz/PeerAuthorizerTest.java +++ b/security-utils/src/test/java/com/yahoo/security/tls/authz/PeerAuthorizerTest.java @@ -158,11 +158,11 @@ public class PeerAuthorizerTest { } private static void assertAuthorized(ConnectionAuthContext result) { - assertTrue(result.succeeded()); + assertTrue(result.authorized()); } private static void assertUnauthorized(ConnectionAuthContext result) { - assertFalse(result.succeeded()); + assertFalse(result.authorized()); } private static void assertCapabiltiesGranted(ConnectionAuthContext ctx, Set<Capability> expected) { |