Browse Source

Using pattern matching for instanceof

pull/13649/head
Seongguk Jeong 3 years ago committed by Josh Cummings
parent
commit
ea19f82b8a
  1. 15
      core/src/main/java/org/springframework/security/authentication/AbstractAuthenticationToken.java
  2. 3
      core/src/main/java/org/springframework/security/authentication/AnonymousAuthenticationToken.java
  3. 3
      web/src/main/java/org/springframework/security/web/jaasapi/JaasApiIntegrationFilter.java

15
core/src/main/java/org/springframework/security/authentication/AbstractAuthenticationToken.java

@ -68,14 +68,14 @@ public abstract class AbstractAuthenticationToken implements Authentication, Cre @@ -68,14 +68,14 @@ public abstract class AbstractAuthenticationToken implements Authentication, Cre
@Override
public String getName() {
if (this.getPrincipal() instanceof UserDetails) {
return ((UserDetails) this.getPrincipal()).getUsername();
if (this.getPrincipal() instanceof UserDetails userDetails) {
return userDetails.getUsername();
}
if (this.getPrincipal() instanceof AuthenticatedPrincipal) {
return ((AuthenticatedPrincipal) this.getPrincipal()).getName();
if (this.getPrincipal() instanceof AuthenticatedPrincipal authenticatedPrincipal) {
return authenticatedPrincipal.getName();
}
if (this.getPrincipal() instanceof Principal) {
return ((Principal) this.getPrincipal()).getName();
if (this.getPrincipal() instanceof Principal principal) {
return principal.getName();
}
return (this.getPrincipal() == null) ? "" : this.getPrincipal().toString();
}
@ -119,10 +119,9 @@ public abstract class AbstractAuthenticationToken implements Authentication, Cre @@ -119,10 +119,9 @@ public abstract class AbstractAuthenticationToken implements Authentication, Cre
@Override
public boolean equals(Object obj) {
if (!(obj instanceof AbstractAuthenticationToken)) {
if (!(obj instanceof AbstractAuthenticationToken test)) {
return false;
}
AbstractAuthenticationToken test = (AbstractAuthenticationToken) obj;
if (!this.authorities.equals(test.authorities)) {
return false;
}

3
core/src/main/java/org/springframework/security/authentication/AnonymousAuthenticationToken.java

@ -74,8 +74,7 @@ public class AnonymousAuthenticationToken extends AbstractAuthenticationToken im @@ -74,8 +74,7 @@ public class AnonymousAuthenticationToken extends AbstractAuthenticationToken im
if (!super.equals(obj)) {
return false;
}
if (obj instanceof AnonymousAuthenticationToken) {
AnonymousAuthenticationToken test = (AnonymousAuthenticationToken) obj;
if (obj instanceof AnonymousAuthenticationToken test) {
return (this.getKeyHash() == test.getKeyHash());
}
return false;

3
web/src/main/java/org/springframework/security/web/jaasapi/JaasApiIntegrationFilter.java

@ -127,10 +127,9 @@ public class JaasApiIntegrationFilter extends GenericFilterBean { @@ -127,10 +127,9 @@ public class JaasApiIntegrationFilter extends GenericFilterBean {
if (!authentication.isAuthenticated()) {
return null;
}
if (!(authentication instanceof JaasAuthenticationToken)) {
if (!(authentication instanceof JaasAuthenticationToken token)) {
return null;
}
JaasAuthenticationToken token = (JaasAuthenticationToken) authentication;
LoginContext loginContext = token.getLoginContext();
if (loginContext == null) {
return null;

Loading…
Cancel
Save