diff --git a/core/spring-security-core.gradle b/core/spring-security-core.gradle index 23fe149d7d..76ef7f2870 100644 --- a/core/spring-security-core.gradle +++ b/core/spring-security-core.gradle @@ -1,6 +1,7 @@ import java.util.concurrent.Callable plugins { + id 'javadoc-warnings-error' id 'security-nullability' } diff --git a/core/src/main/java/org/springframework/security/access/hierarchicalroles/RoleHierarchyUtils.java b/core/src/main/java/org/springframework/security/access/hierarchicalroles/RoleHierarchyUtils.java index 5db6efb028..1b77aef165 100644 --- a/core/src/main/java/org/springframework/security/access/hierarchicalroles/RoleHierarchyUtils.java +++ b/core/src/main/java/org/springframework/security/access/hierarchicalroles/RoleHierarchyUtils.java @@ -36,14 +36,14 @@ public final class RoleHierarchyUtils { /** * Converts the supplied {@link Map} of role name to implied role name(s) to a string - * representation understood by {@link RoleHierarchyImpl#setHierarchy(String)}. The + * representation understood by {@code RoleHierarchyImpl#setHierarchy(String)}. The * map key is the role name and the map value is a {@link List} of implied role * name(s). * @param roleHierarchyMap the mapping(s) of role name to implied role name(s) * @return a string representation of a role hierarchy * @throws IllegalArgumentException if roleHierarchyMap is null or empty or if a role * name is null or empty or if an implied role name(s) is null or empty - * @deprecated please see {@link RoleHierarchyImpl#setHierarchy} deprecation notice + * @deprecated please see {@code RoleHierarchyImpl#setHierarchy} deprecation notice */ @Deprecated public static String roleHierarchyFromMap(Map> roleHierarchyMap) { diff --git a/core/src/main/java/org/springframework/security/access/package-info.java b/core/src/main/java/org/springframework/security/access/package-info.java index 01ac014fe7..d9b0f81f0d 100644 --- a/core/src/main/java/org/springframework/security/access/package-info.java +++ b/core/src/main/java/org/springframework/security/access/package-info.java @@ -18,7 +18,7 @@ * Core access-control related code, including security metadata related classes, * interception code, access control annotations, EL support and voter-based * implementations of the central - * {@link org.springframework.security.access.AccessDecisionManager AccessDecisionManager} + * {@code AccessDecisionManager} * interface. */ @NullMarked diff --git a/core/src/main/java/org/springframework/security/authentication/AuthenticationTrustResolver.java b/core/src/main/java/org/springframework/security/authentication/AuthenticationTrustResolver.java index fba8f77cb2..e16eb59fa3 100644 --- a/core/src/main/java/org/springframework/security/authentication/AuthenticationTrustResolver.java +++ b/core/src/main/java/org/springframework/security/authentication/AuthenticationTrustResolver.java @@ -77,7 +77,7 @@ public interface AuthenticationTrustResolver { * Checks if the {@link Authentication} is not null, authenticated, and not anonymous. * @param authentication the {@link Authentication} to check. * @return true if the {@link Authentication} is not null, - * {@link #isAnonymous(Authentication)} returns false, & + * {@link #isAnonymous(Authentication)} returns false, and * {@link Authentication#isAuthenticated()} is true. * @since 6.1.7 */ diff --git a/core/src/main/java/org/springframework/security/core/SpringSecurityCoreVersion.java b/core/src/main/java/org/springframework/security/core/SpringSecurityCoreVersion.java index 726db080f3..ec93a6bcda 100644 --- a/core/src/main/java/org/springframework/security/core/SpringSecurityCoreVersion.java +++ b/core/src/main/java/org/springframework/security/core/SpringSecurityCoreVersion.java @@ -42,7 +42,7 @@ public final class SpringSecurityCoreVersion { /** * Global Serialization value for Spring Security classes. * @deprecated Please have each class use its own serialization version - * @see SpringSecurityCoreVersionSerializableTests + * For more details, refer to the {@code SpringSecurityCoreVersionSerializableTests} class. */ @Deprecated(forRemoval = true) public static final long SERIAL_VERSION_UID = 620L; diff --git a/core/src/main/java/org/springframework/security/core/annotation/SecurityAnnotationScanner.java b/core/src/main/java/org/springframework/security/core/annotation/SecurityAnnotationScanner.java index 441c9c4a01..311204314c 100644 --- a/core/src/main/java/org/springframework/security/core/annotation/SecurityAnnotationScanner.java +++ b/core/src/main/java/org/springframework/security/core/annotation/SecurityAnnotationScanner.java @@ -72,9 +72,9 @@ public interface SecurityAnnotationScanner { * attributable to the parameter. * *

- * Implementations should describe their strategy for searching the element and any + * Implementations should describe their strategy for searching the parameter and any * surrounding class, interfaces, or super-class. - * @param element the element to search + * @param parameter the parameter to search * @return the synthesized annotation or {@code null} if not found */ @Nullable A scan(Parameter parameter);