You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shiro.apache.org by lp...@apache.org on 2023/07/31 21:28:54 UTC
[shiro] branch dependabot/maven/com.puppycrawl.tools-checkstyle-10.12.2 updated: enh(checkstyle): made private classes final to conform to latest checkstyle rules
This is an automated email from the ASF dual-hosted git repository.
lprimak pushed a commit to branch dependabot/maven/com.puppycrawl.tools-checkstyle-10.12.2
in repository https://gitbox.apache.org/repos/asf/shiro.git
The following commit(s) were added to refs/heads/dependabot/maven/com.puppycrawl.tools-checkstyle-10.12.2 by this push:
new 4728b6be7 enh(checkstyle): made private classes final to conform to latest checkstyle rules
4728b6be7 is described below
commit 4728b6be751b5774c7fa197ffc228d4fa5e09df2
Author: lprimak <le...@flowlogix.com>
AuthorDate: Mon Jul 31 16:28:15 2023 -0500
enh(checkstyle): made private classes final to conform to latest checkstyle rules
---
.../java/org/apache/shiro/cdi/AnnotatedTypeWrapperTest.java | 8 ++++----
.../org/apache/shiro/ee/cdi/ShiroSessionScopeExtension.java | 12 ++++++------
.../apache/shiro/ee/filters/FormAuthenticationFilter.java | 2 +-
.../main/java/org/apache/shiro/ee/filters/LogoutFilter.java | 2 +-
.../shiro/ee/filters/PassThruAuthenticationFilter.java | 2 +-
.../java/org/apache/shiro/ee/listeners/IniEnvironment.java | 2 +-
.../java/org/apache/shiro/ee/cdi/ShiroScopeContextTest.java | 2 +-
7 files changed, 15 insertions(+), 15 deletions(-)
diff --git a/support/cdi/src/test/java/org/apache/shiro/cdi/AnnotatedTypeWrapperTest.java b/support/cdi/src/test/java/org/apache/shiro/cdi/AnnotatedTypeWrapperTest.java
index c643bdc41..917cee4d8 100644
--- a/support/cdi/src/test/java/org/apache/shiro/cdi/AnnotatedTypeWrapperTest.java
+++ b/support/cdi/src/test/java/org/apache/shiro/cdi/AnnotatedTypeWrapperTest.java
@@ -44,17 +44,17 @@ class AnnotatedTypeWrapperTest {
@RequiresAuthentication
@RequiresGuest
@RequiresPermissions("hello")
- private class Annotated { }
+ private final class Annotated { }
@ShiroSecureAnnotation
- private class ShiroSecureAnnotated { }
+ private final class ShiroSecureAnnotated { }
@Stateless
- private class StatelessAnnotated { }
+ private final class StatelessAnnotated { }
@SessionScoped
@SuppressWarnings("serial")
- private static class SessionScopedAnnotated implements Serializable { }
+ private static final class SessionScopedAnnotated implements Serializable { }
@Test
void noAnnotations() {
diff --git a/support/jakarta-ee/src/main/java/org/apache/shiro/ee/cdi/ShiroSessionScopeExtension.java b/support/jakarta-ee/src/main/java/org/apache/shiro/ee/cdi/ShiroSessionScopeExtension.java
index dbadd6b61..9ad3386e9 100644
--- a/support/jakarta-ee/src/main/java/org/apache/shiro/ee/cdi/ShiroSessionScopeExtension.java
+++ b/support/jakarta-ee/src/main/java/org/apache/shiro/ee/cdi/ShiroSessionScopeExtension.java
@@ -46,23 +46,23 @@ public class ShiroSessionScopeExtension implements Extension, Serializable {
@SessionScoped
@SuppressWarnings("serial")
- private static class SessionScopedAnnotated implements Serializable { }
+ private static final class SessionScopedAnnotated implements Serializable { }
@javax.faces.view.ViewScoped
@SuppressWarnings("serial")
- private static class FacesViewScopedAnnotated implements Serializable { }
+ private static final class FacesViewScopedAnnotated implements Serializable { }
@org.omnifaces.cdi.ViewScoped
@SuppressWarnings("serial")
- private static class OmniViewScopedAnnotated implements Serializable { }
+ private static final class OmniViewScopedAnnotated implements Serializable { }
@ShiroSessionScoped
@SuppressWarnings("serial")
- private static class ShiroSessionScopedAnnotated implements Serializable { }
+ private static final class ShiroSessionScopedAnnotated implements Serializable { }
@ShiroFacesViewScoped
@SuppressWarnings("serial")
- private static class ShiroFacesViewScopedAnnotated implements Serializable { }
+ private static final class ShiroFacesViewScopedAnnotated implements Serializable { }
@ShiroOmniViewScoped
@SuppressWarnings("serial")
- private static class ShiroOmniViewScopedAnnotated implements Serializable { }
+ private static final class ShiroOmniViewScopedAnnotated implements Serializable { }
/**
diff --git a/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/FormAuthenticationFilter.java b/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/FormAuthenticationFilter.java
index 0dc7582d7..a65b4c0dc 100644
--- a/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/FormAuthenticationFilter.java
+++ b/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/FormAuthenticationFilter.java
@@ -38,7 +38,7 @@ public class FormAuthenticationFilter extends org.apache.shiro.web.filter.authc.
static final FallbackPredicate NO_PREDICATE = (path, request) -> false;
private final @Delegate AuthenticationFilterDelegate delegate;
- private class Methods implements MethodsFromFilter {
+ private final class Methods implements MethodsFromFilter {
@Override
public Subject getSubject(ServletRequest request, ServletResponse response) {
return FormAuthenticationFilter.super.getSubject(request, response);
diff --git a/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/LogoutFilter.java b/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/LogoutFilter.java
index 06d43dd11..ed54b7731 100644
--- a/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/LogoutFilter.java
+++ b/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/LogoutFilter.java
@@ -33,7 +33,7 @@ public class LogoutFilter extends org.apache.shiro.web.filter.authc.LogoutFilter
static final String LOGOUT_PREDICATE_ATTR_NAME = "org.apache.shiro.ee.logout-predicate";
private final @Delegate AuthenticationFilterDelegate delegate;
- private class Methods implements MethodsFromFilter {
+ private final class Methods implements MethodsFromFilter {
@Override
public Subject getSubject(ServletRequest request, ServletResponse response) {
return LogoutFilter.super.getSubject(request, response);
diff --git a/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/PassThruAuthenticationFilter.java b/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/PassThruAuthenticationFilter.java
index 0a9402d21..00318874c 100644
--- a/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/PassThruAuthenticationFilter.java
+++ b/support/jakarta-ee/src/main/java/org/apache/shiro/ee/filters/PassThruAuthenticationFilter.java
@@ -30,7 +30,7 @@ import org.apache.shiro.subject.Subject;
public class PassThruAuthenticationFilter extends org.apache.shiro.web.filter.authc.PassThruAuthenticationFilter {
private final @Delegate AuthenticationFilterDelegate delegate;
- private class Methods implements MethodsFromFilter {
+ private final class Methods implements MethodsFromFilter {
@Override
public Subject getSubject(ServletRequest request, ServletResponse response) {
return PassThruAuthenticationFilter.super.getSubject(request, response);
diff --git a/support/jakarta-ee/src/main/java/org/apache/shiro/ee/listeners/IniEnvironment.java b/support/jakarta-ee/src/main/java/org/apache/shiro/ee/listeners/IniEnvironment.java
index 512a1399b..d38ef7dff 100644
--- a/support/jakarta-ee/src/main/java/org/apache/shiro/ee/listeners/IniEnvironment.java
+++ b/support/jakarta-ee/src/main/java/org/apache/shiro/ee/listeners/IniEnvironment.java
@@ -40,7 +40,7 @@ public class IniEnvironment extends IniWebEnvironment {
private String otherConfigLocation;
@SuppressWarnings("deprecation")
- private static class SecurityManagerFactory extends WebIniSecurityManagerFactory {
+ private static final class SecurityManagerFactory extends WebIniSecurityManagerFactory {
private final Lazy<AesCipherService> cipherService = new Lazy<>(AesCipherService::new);
@Override
diff --git a/support/jakarta-ee/src/test/java/org/apache/shiro/ee/cdi/ShiroScopeContextTest.java b/support/jakarta-ee/src/test/java/org/apache/shiro/ee/cdi/ShiroScopeContextTest.java
index 3241ea123..003a6408f 100644
--- a/support/jakarta-ee/src/test/java/org/apache/shiro/ee/cdi/ShiroScopeContextTest.java
+++ b/support/jakarta-ee/src/test/java/org/apache/shiro/ee/cdi/ShiroScopeContextTest.java
@@ -56,7 +56,7 @@ public class ShiroScopeContextTest {
private MyBean bean;
private ShiroScopeContext ctx;
- private static class MyBean implements ViewScoped, Serializable {
+ private static final class MyBean implements ViewScoped, Serializable {
private static final long serialVersionUID = 1L;
@Override