[java-plugin-shibd] 01/02: Rationalize names of Storage-related properties.

[Codeberg]

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

codeberg pushed a commit to branch main
in repository java-plugin-shibd.

View the commit online:
https://codeberg.org/Shibboleth/java-plugin-shibd/commit/fe217b49913bab2d5943d55100abc6f582c42ac8

commit fe217b49913bab2d5943d55100abc6f582c42ac8
Author: Scott Cantor <scott at restingparrotsoftware.com>
AuthorDate: Wed Oct 21 14:01:38 2026 -0400

    Rationalize names of Storage-related properties.
---
 .../shibboleth/idp/flows/sp/session-cache/session-cache-beans.xml    | 2 +-
 .../META-INF/net/shibboleth/idp/flows/sp/storage/storage-beans.xml   | 2 +-
 .../main/resources/net/shibboleth/idp/module/conf/sp/sp.properties   | 5 ++++-
 .../sp/flows/TestSPEnvironmentApplicationContextInitializer.java     | 5 +++--
 4 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/sp-conf-impl/src/main/resources/META-INF/net/shibboleth/idp/flows/sp/session-cache/session-cache-beans.xml b/sp-conf-impl/src/main/resources/META-INF/net/shibboleth/idp/flows/sp/session-cache/session-cache-beans.xml
index b494b82..636d216 100644
--- a/sp-conf-impl/src/main/resources/META-INF/net/shibboleth/idp/flows/sp/session-cache/session-cache-beans.xml
+++ b/sp-conf-impl/src/main/resources/META-INF/net/shibboleth/idp/flows/sp/session-cache/session-cache-beans.xml
@@ -15,6 +15,6 @@
     <bean id="DoSessionCacheOperation"
         class="net.shibboleth.sp.profile.impl.DoSessionCacheOperation" scope="prototype"
         p:maxStorageTimeout="%{sp.session.maxStorageTimeout:P1D}"
-        p:storageService-ref="#{'%{sp.session.storageService:shibboleth.StorageService}'.trim()}" />
+        p:storageService-ref="#{'%{sp.session.StorageService:shibboleth.StorageService}'.trim()}" />
 
 </beans>
diff --git a/sp-conf-impl/src/main/resources/META-INF/net/shibboleth/idp/flows/sp/storage/storage-beans.xml b/sp-conf-impl/src/main/resources/META-INF/net/shibboleth/idp/flows/sp/storage/storage-beans.xml
index e31d261..eea540b 100644
--- a/sp-conf-impl/src/main/resources/META-INF/net/shibboleth/idp/flows/sp/storage/storage-beans.xml
+++ b/sp-conf-impl/src/main/resources/META-INF/net/shibboleth/idp/flows/sp/storage/storage-beans.xml
@@ -16,6 +16,6 @@
     <bean id="DoStorageOperation"
         class="net.shibboleth.sp.profile.impl.DoStorageOperation" scope="prototype"
         p:httpServletRequestSupplier-ref="shibboleth.HttpServletRequestSupplier"
-        p:storageService-ref="#{'%{sp.storageService:shibboleth.StorageService}'.trim()}" />
+        p:storageService-ref="#{'%{sp.StorageService:shibboleth.StorageService}'.trim()}" />
 
 </beans>
diff --git a/sp-conf-impl/src/main/resources/net/shibboleth/idp/module/conf/sp/sp.properties b/sp-conf-impl/src/main/resources/net/shibboleth/idp/module/conf/sp/sp.properties
index d1c17eb..4930aa4 100644
--- a/sp-conf-impl/src/main/resources/net/shibboleth/idp/module/conf/sp/sp.properties
+++ b/sp-conf-impl/src/main/resources/net/shibboleth/idp/module/conf/sp/sp.properties
@@ -62,8 +62,11 @@ sp.service.credential.checkInterval = PT5M
 #sp.discoveryURL =
 #sp.discoveryURLFunction =
 
+# Enables revocation support for sessions
+#sp.revocation.StorageService =
+
 # Settings for Hub-mediated session storage and encryption features
-#sp.session.storageService = shibboleth.StorageService
+#sp.session.StorageService = shibboleth.StorageService
 #sp.session.maxStorageTimeout = P1D
 #sp.dataSealer = shibboleth.DataSealer
 
diff --git a/sp-conf-impl/src/test/java/net/shibboleth/sp/flows/TestSPEnvironmentApplicationContextInitializer.java b/sp-conf-impl/src/test/java/net/shibboleth/sp/flows/TestSPEnvironmentApplicationContextInitializer.java
index 285d0ff..0f30c62 100644
--- a/sp-conf-impl/src/test/java/net/shibboleth/sp/flows/TestSPEnvironmentApplicationContextInitializer.java
+++ b/sp-conf-impl/src/test/java/net/shibboleth/sp/flows/TestSPEnvironmentApplicationContextInitializer.java
@@ -42,8 +42,9 @@ public class TestSPEnvironmentApplicationContextInitializer extends TestEnvironm
         // Set to allow disco flow to run.
         mock.setProperty("sp.discoveryURL", "https://ds.example.org/DS");
         
-        mock.setProperty("sp.storageService", "shibboleth.StorageService");
-        mock.setProperty("sp.session.storageService", "shibboleth.StorageService");
+        mock.setProperty("sp.StorageService", "shibboleth.StorageService");
+        mock.setProperty("sp.session.StorageService", "shibboleth.StorageService");
+        mock.setProperty("sp.revocation.StorageService", "shibboleth.StorageService");
         mock.setProperty("idp.additionalProperties",
                 "/conf/ldap.properties, /conf/saml-nameid.properties, /conf/services.properties, /conf/admin/admin.properties, /conf/authn/authn.properties, /conf/c14n/subject-c14n.properties, /credentials/secrets.properties, /conf/sp/sp.properties");
     }

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