[java-shib-profile] branch main updated: Add missing precondition checks.

[Scott Cantor]

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

scantor pushed a commit to branch main
in repository java-shib-profile.

View the commit online:
http://git.shibboleth.net/view/?p=java-shib-profile.git;a=commit;h=be984d76c37dd1c0aa257f9aee76fbd4a61bcfcd

The following commit(s) were added to refs/heads/main by this push:
     new be984d7  Add missing precondition checks.
be984d7 is described below

commit be984d76c37dd1c0aa257f9aee76fbd4a61bcfcd
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Thu Sep 12 14:17:19 2024 -0400

    Add missing precondition checks.
---
 .../profile/relyingparty/BasicRelyingPartyConfiguration.java   | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/shib-profile-api/src/main/java/net/shibboleth/profile/relyingparty/BasicRelyingPartyConfiguration.java b/shib-profile-api/src/main/java/net/shibboleth/profile/relyingparty/BasicRelyingPartyConfiguration.java
index bcb44f7..059698d 100644
--- a/shib-profile-api/src/main/java/net/shibboleth/profile/relyingparty/BasicRelyingPartyConfiguration.java
+++ b/shib-profile-api/src/main/java/net/shibboleth/profile/relyingparty/BasicRelyingPartyConfiguration.java
@@ -89,6 +89,7 @@ public class BasicRelyingPartyConfiguration extends AbstractIdentifiableInitiali
      */
     public void setIssuer(@Nullable @NotEmpty final String issuer) {
         checkSetterPreconditions();
+        
         issuerLookupStrategy = FunctionSupport.constant(issuer);
     }
 
@@ -101,6 +102,7 @@ public class BasicRelyingPartyConfiguration extends AbstractIdentifiableInitiali
      */
     public void setIssuerLookupStrategy(@Nonnull final Function<ProfileRequestContext,String> strategy) {
         checkSetterPreconditions();
+        
         issuerLookupStrategy = Constraint.isNotNull(strategy, "Lookup strategy cannot be null");
     }
     
@@ -137,6 +139,7 @@ public class BasicRelyingPartyConfiguration extends AbstractIdentifiableInitiali
      */
     public void setDetailedErrors(final boolean flag) {
         checkSetterPreconditions();
+        
         detailedErrorsPredicate = PredicateSupport.constant(flag);
     }
     
@@ -146,6 +149,8 @@ public class BasicRelyingPartyConfiguration extends AbstractIdentifiableInitiali
      * @param condition  condition to set
      */
     public void setDetailedErrorsPredicate(@Nonnull final Predicate<ProfileRequestContext> condition) {
+        checkSetterPreconditions();
+        
         detailedErrorsPredicate = Constraint.isNotNull(condition, "Condition cannot be null");
     }
     
@@ -163,6 +168,8 @@ public class BasicRelyingPartyConfiguration extends AbstractIdentifiableInitiali
      * @since 5.0.0
      */
     public void setSecurityConfiguration(@Nullable final SecurityConfiguration configuration) {
+        checkSetterPreconditions();
+        
         securityConfigurationLookupStrategy = FunctionSupport.constant(configuration);
     }
 
@@ -207,6 +214,7 @@ public class BasicRelyingPartyConfiguration extends AbstractIdentifiableInitiali
      */
     public void setProfileConfigurations(@Nullable final Collection<ProfileConfiguration> configs) {
         checkSetterPreconditions();
+        
         if (configs == null) {
             profileConfigurationsLookupStrategy = FunctionSupport.constant(null);
         } else {
@@ -231,6 +239,7 @@ public class BasicRelyingPartyConfiguration extends AbstractIdentifiableInitiali
     public void setProfileConfigurationsLookupStrategy(
             @Nonnull final Function<ProfileRequestContext,Map<String,ProfileConfiguration>> strategy) {
         checkSetterPreconditions();
+        
         profileConfigurationsLookupStrategy = Constraint.isNotNull(strategy, "Lookup strategy cannot be null");
     }
 
@@ -241,6 +250,7 @@ public class BasicRelyingPartyConfiguration extends AbstractIdentifiableInitiali
      */
     public void setActivationCondition(@Nonnull final Predicate<ProfileRequestContext> condition) {
         checkSetterPreconditions();
+        
         activationCondition =
                 Constraint.isNotNull(condition, "Relying party configuration activation condition cannot be null");
     }

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