Skip to content

Commit 981e3fd

Browse files
committed
Merge branch '6.4.x'
2 parents 11113ad + b4c7795 commit 981e3fd

9 files changed

+23
-1
lines changed

config/src/test/java/org/springframework/security/SpringSecurityCoreVersionSerializableTests.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,9 @@
9999
import org.springframework.security.authentication.ott.InvalidOneTimeTokenException;
100100
import org.springframework.security.authentication.ott.OneTimeTokenAuthenticationToken;
101101
import org.springframework.security.authentication.password.CompromisedPasswordException;
102+
import org.springframework.security.authorization.AuthorityAuthorizationDecision;
103+
import org.springframework.security.authorization.AuthorizationDecision;
104+
import org.springframework.security.authorization.AuthorizationDeniedException;
102105
import org.springframework.security.cas.authentication.CasAssertionAuthenticationToken;
103106
import org.springframework.security.cas.authentication.CasAuthenticationToken;
104107
import org.springframework.security.cas.authentication.CasServiceTicketAuthenticationToken;
@@ -471,6 +474,11 @@ class SpringSecurityCoreVersionSerializableTests {
471474
generatorByClassName.put(AbstractSessionEvent.class, (r) -> new AbstractSessionEvent(securityContext));
472475
generatorByClassName.put(SecurityConfig.class, (r) -> new SecurityConfig("value"));
473476
generatorByClassName.put(TransientSecurityContext.class, (r) -> new TransientSecurityContext(authentication));
477+
generatorByClassName.put(AuthorizationDeniedException.class,
478+
(r) -> new AuthorizationDeniedException("message", new AuthorizationDecision(false)));
479+
generatorByClassName.put(AuthorizationDecision.class, (r) -> new AuthorizationDecision(true));
480+
generatorByClassName.put(AuthorityAuthorizationDecision.class,
481+
(r) -> new AuthorityAuthorizationDecision(true, AuthorityUtils.createAuthorityList("ROLE_USER")));
474482

475483
// cas
476484
generatorByClassName.put(CasServiceTicketAuthenticationToken.class, (r) -> {

config/src/test/resources/serialized/6.4.x/org.springframework.security.authorization.AuthorizationDeniedException.serialized

Whitespace-only changes.

core/src/main/java/org/springframework/security/authorization/AuthorityAuthorizationDecision.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616

1717
package org.springframework.security.authorization;
1818

19+
import java.io.Serial;
1920
import java.util.Collection;
2021

2122
import org.springframework.security.core.GrantedAuthority;
@@ -28,6 +29,9 @@
2829
*/
2930
public class AuthorityAuthorizationDecision extends AuthorizationDecision {
3031

32+
@Serial
33+
private static final long serialVersionUID = -8338309042331376592L;
34+
3135
private final Collection<GrantedAuthority> authorities;
3236

3337
public AuthorityAuthorizationDecision(boolean granted, Collection<GrantedAuthority> authorities) {

core/src/main/java/org/springframework/security/authorization/AuthorizationDecision.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,17 @@
1616

1717
package org.springframework.security.authorization;
1818

19+
import java.io.Serial;
20+
1921
/**
2022
* @author Rob Winch
2123
* @since 5.0
2224
*/
2325
public class AuthorizationDecision implements AuthorizationResult {
2426

27+
@Serial
28+
private static final long serialVersionUID = -3226018324649244416L;
29+
2530
private final boolean granted;
2631

2732
public AuthorizationDecision(boolean granted) {

core/src/main/java/org/springframework/security/authorization/AuthorizationManagers.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -145,6 +145,7 @@ public static <T> AuthorizationManager<T> not(AuthorizationManager<T> manager) {
145145
private AuthorizationManagers() {
146146
}
147147

148+
@SuppressWarnings("serial")
148149
private static final class CompositeAuthorizationDecision extends AuthorizationDecision {
149150

150151
private final List<AuthorizationResult> results;
@@ -161,6 +162,7 @@ public String toString() {
161162

162163
}
163164

165+
@SuppressWarnings("serial")
164166
private static final class NotAuthorizationDecision extends AuthorizationDecision {
165167

166168
private final AuthorizationResult result;

core/src/main/java/org/springframework/security/authorization/AuthorizationResult.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,15 @@
1616

1717
package org.springframework.security.authorization;
1818

19+
import java.io.Serializable;
20+
1921
/**
2022
* Represents an authorization result
2123
*
2224
* @author Marcus da Coregio
2325
* @since 6.3
2426
*/
25-
public interface AuthorizationResult {
27+
public interface AuthorizationResult extends Serializable {
2628

2729
/**
2830
* @return whether the access has been granted

core/src/main/java/org/springframework/security/authorization/ExpressionAuthorizationDecision.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
* @author Marcus Da Coregio
2525
* @since 5.8
2626
*/
27+
@SuppressWarnings("serial")
2728
public class ExpressionAuthorizationDecision extends AuthorizationDecision {
2829

2930
private final Expression expression;

0 commit comments

Comments
 (0)