Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
@ExtendWith(ArquillianExtension.class)
public class NoIniJaxRsIT {
@Builder
private static class Credentials {
private static final class Credentials {
final boolean send;
final String username;
final String password;
Expand Down
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -557,7 +557,7 @@
<dependency>
<groupId>com.puppycrawl.tools</groupId>
<artifactId>checkstyle</artifactId>
<version>10.12.1</version>
<version>10.12.2</version>
</dependency>
</dependencies>
</plugin>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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 { }


/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
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);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down