[java-identity-provider] branch main updated: Javadoc issues and other warnings.
Scott Cantor
cantor.2 at osu.edu
Thu Feb 9 20:20:07 UTC 2023
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=6297f9dd68ade5623c7aec2b00c54affc2639509
The following commit(s) were added to refs/heads/main by this push:
new 6297f9dd6 Javadoc issues and other warnings.
6297f9dd6 is described below
commit 6297f9dd68ade5623c7aec2b00c54affc2639509
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Thu Feb 9 15:20:04 2023 -0500
Javadoc issues and other warnings.
---
.../main/java/net/shibboleth/idp/profile/IdPEventIds.java | 13 ++++++++-----
.../CriteriaRelyingPartyConfigurationResolver.java | 5 +++--
.../idp/relyingparty/RelyingPartyConfiguration.java | 6 +++---
.../idp/relyingparty/RelyingPartyConfigurationResolver.java | 5 +++--
4 files changed, 17 insertions(+), 12 deletions(-)
diff --git a/idp-profile-api/src/main/java/net/shibboleth/idp/profile/IdPEventIds.java b/idp-profile-api/src/main/java/net/shibboleth/idp/profile/IdPEventIds.java
index 7f1eddc86..c2f2bd140 100644
--- a/idp-profile-api/src/main/java/net/shibboleth/idp/profile/IdPEventIds.java
+++ b/idp-profile-api/src/main/java/net/shibboleth/idp/profile/IdPEventIds.java
@@ -19,6 +19,9 @@ package net.shibboleth.idp.profile;
import javax.annotation.Nonnull;
+import net.shibboleth.idp.profile.context.RelyingPartyContext;
+import net.shibboleth.idp.relyingparty.RelyingPartyConfiguration;
+import net.shibboleth.profile.config.ProfileConfiguration;
import net.shibboleth.shared.annotation.constraint.NotEmpty;
// TODO: move attribute related events into that module?
@@ -30,28 +33,28 @@ import net.shibboleth.shared.annotation.constraint.NotEmpty;
public final class IdPEventIds {
/**
- * ID of event returned if the {@link net.shibboleth.idp.profile.context.RelyingPartyContext} is missing or corrupt.
+ * ID of event returned if the {@link RelyingPartyContext} is missing or corrupt.
*/
@Nonnull @NotEmpty public static final String INVALID_RELYING_PARTY_CTX = "InvalidRelyingPartyContext";
/**
- * ID of event returned if the {@link net.shibboleth.idp.relyingparty.RelyingPartyConfiguration} is missing or
+ * ID of event returned if the {@link RelyingPartyConfiguration} is missing or
* corrupt.
*/
@Nonnull @NotEmpty public static final String INVALID_RELYING_PARTY_CONFIG = "InvalidRelyingPartyConfiguration";
/**
- * ID of event returned if the {@link net.shibboleth.idp.profile.config.ProfileConfiguration} is missing or corrupt.
+ * ID of event returned if the {@link ProfileConfiguration} is missing or corrupt.
*/
@Nonnull @NotEmpty public static final String INVALID_PROFILE_CONFIG = "InvalidProfileConfiguration";
/**
- * ID of event returned if the @link net.shibboleth.idp.attribute.context.AttributeContext is missing or corrupt.
+ * ID of event returned if the @link AttributeContext is missing or corrupt.
*/
@Nonnull @NotEmpty public static final String INVALID_ATTRIBUTE_CTX = "InvalidAttributeContext";
/**
- * ID of event returned if the @link net.shibboleth.idp.authn.context.SubjectContext is missing or corrupt.
+ * ID of event returned if the @link SubjectContext is missing or corrupt.
*/
@Nonnull @NotEmpty public static final String INVALID_SUBJECT_CTX = "InvalidSubjectContext";
diff --git a/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/CriteriaRelyingPartyConfigurationResolver.java b/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/CriteriaRelyingPartyConfigurationResolver.java
index 8528ea9e5..301005b7b 100644
--- a/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/CriteriaRelyingPartyConfigurationResolver.java
+++ b/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/CriteriaRelyingPartyConfigurationResolver.java
@@ -22,6 +22,7 @@ import javax.annotation.Nullable;
import org.opensaml.security.config.SecurityConfiguration;
+import net.shibboleth.profile.config.ProfileConfiguration;
import net.shibboleth.shared.annotation.constraint.NotEmpty;
import net.shibboleth.shared.component.IdentifiedComponent;
import net.shibboleth.shared.resolver.CriteriaSet;
@@ -34,8 +35,8 @@ public interface CriteriaRelyingPartyConfigurationResolver extends Resolver<Rely
/**
* Return the default security configuration for the profile.
*
- * @param profileId the profile ID (available via
- * {@link net.shibboleth.idp.profile.config.ProfileConfiguration#getId()}
+ * @param profileId the profile ID (available via {@link ProfileConfiguration#getId()}
+ *
* @return the configured default configuration
*/
@Nullable SecurityConfiguration getDefaultSecurityConfiguration(@Nonnull @NotEmpty final String profileId);
diff --git a/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/RelyingPartyConfiguration.java b/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/RelyingPartyConfiguration.java
index e3f2d2b0c..9f3129ca0 100644
--- a/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/RelyingPartyConfiguration.java
+++ b/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/RelyingPartyConfiguration.java
@@ -18,7 +18,6 @@
package net.shibboleth.idp.relyingparty;
import java.util.Collection;
-import java.util.Collections;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -36,6 +35,7 @@ import net.shibboleth.shared.annotation.constraint.NonnullElements;
import net.shibboleth.shared.annotation.constraint.NotEmpty;
import net.shibboleth.shared.annotation.constraint.NotLive;
import net.shibboleth.shared.annotation.constraint.Unmodifiable;
+import net.shibboleth.shared.collection.CollectionSupport;
import net.shibboleth.shared.component.AbstractIdentifiableInitializableComponent;
import net.shibboleth.shared.component.ComponentInitializationException;
import net.shibboleth.shared.component.IdentifiedComponent;
@@ -146,9 +146,9 @@ public class RelyingPartyConfiguration extends AbstractIdentifiableInitializable
final Map<String,ProfileConfiguration> map = profileConfigurationsLookupStrategy.apply(profileRequestContext);
if (map != null) {
- return Map.copyOf(map);
+ return CollectionSupport.copyToMap(map);
}
- return Collections.emptyMap();
+ return CollectionSupport.emptyMap();
}
/**
diff --git a/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/RelyingPartyConfigurationResolver.java b/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/RelyingPartyConfigurationResolver.java
index 5db68ee61..41f7c3e98 100644
--- a/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/RelyingPartyConfigurationResolver.java
+++ b/idp-profile-api/src/main/java/net/shibboleth/idp/relyingparty/RelyingPartyConfigurationResolver.java
@@ -20,6 +20,7 @@ package net.shibboleth.idp.relyingparty;
import javax.annotation.Nonnull;
import javax.annotation.Nullable;
+import net.shibboleth.profile.config.ProfileConfiguration;
import net.shibboleth.shared.annotation.constraint.NotEmpty;
import net.shibboleth.shared.component.IdentifiedComponent;
import net.shibboleth.shared.resolver.Resolver;
@@ -34,8 +35,8 @@ public interface RelyingPartyConfigurationResolver extends Resolver<RelyingParty
/**
* Return the default security configuration for the profile.
*
- * @param profileId the profile ID (available via
- * {@link net.shibboleth.idp.profile.config.ProfileConfiguration#getId()}
+ * @param profileId the profile ID (available via {@link ProfileConfiguration#getId()}
+ *
* @return the configured default configuration
*/
@Nullable SecurityConfiguration getDefaultSecurityConfiguration(@Nonnull @NotEmpty final String profileId);
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.
More information about the commits
mailing list