[java-identity-provider] branch main updated: Collapse storage caps interface.

Scott Cantor cantor.2 at osu.edu
Mon Aug 8 14:48:42 UTC 2022


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

scantor pushed a commit to branch main
in repository java-identity-provider.

View the commit online:
http://git.shibboleth.net/view/?p=java-identity-provider.git;a=commit;h=ea52d0efe0b6da8baea80dbd11af77a35dea1f68

The following commit(s) were added to refs/heads/main by this push:
     new ea52d0efe Collapse storage caps interface.
ea52d0efe is described below

commit ea52d0efe0b6da8baea80dbd11af77a35dea1f68
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Mon Aug 8 10:48:39 2022 -0400

    Collapse storage caps interface.
---
 .../idp/authn/impl/StorageBackedAccountLockoutManager.java       | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/StorageBackedAccountLockoutManager.java b/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/StorageBackedAccountLockoutManager.java
index ed9575632..b767d1709 100644
--- a/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/StorageBackedAccountLockoutManager.java
+++ b/idp-authn-impl/src/main/java/net/shibboleth/idp/authn/impl/StorageBackedAccountLockoutManager.java
@@ -27,7 +27,6 @@ import javax.annotation.Nullable;
 
 import org.opensaml.profile.context.ProfileRequestContext;
 import org.opensaml.storage.StorageCapabilities;
-import org.opensaml.storage.StorageCapabilitiesEx;
 import org.opensaml.storage.StorageRecord;
 import org.opensaml.storage.StorageService;
 import org.slf4j.Logger;
@@ -91,11 +90,9 @@ public class StorageBackedAccountLockoutManager extends AbstractIdentifiableInit
         checkSetterPreconditions();
         storageService = Constraint.isNotNull(storage, "StorageService cannot be null");
         final StorageCapabilities caps = storageService.getCapabilities();
-        if (caps instanceof StorageCapabilitiesEx) {
-            Constraint.isTrue(((StorageCapabilitiesEx) caps).isServerSide(), "StorageService cannot be client-side");
-            if (!((StorageCapabilitiesEx) caps).isClustered()) {
-                log.info("Use of non-clustered storage service will result in per-node lockout behavior");
-            }
+        Constraint.isTrue(caps.isServerSide(), "StorageService cannot be client-side");
+        if (!caps.isClustered()) {
+            log.info("Use of non-clustered storage service will result in per-node lockout behavior");
         }
     }
 

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


More information about the commits mailing list