Skip to content

Commit f9e0459

Browse files
plll0123jzheaux
authored andcommitted
Refactor authorization manager variable naming
- Renamed PERMIT_ALL_AUTHORIZATION_MANAGER to snake_case style - Introduced AUTHORIZATION_DECISION for reuse Signed-off-by: plll0123 <[email protected]>
1 parent 92c2e21 commit f9e0459

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

config/src/main/java/org/springframework/security/config/annotation/web/configurers/AuthorizeHttpRequestsConfigurer.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,10 @@
5757
public final class AuthorizeHttpRequestsConfigurer<H extends HttpSecurityBuilder<H>>
5858
extends AbstractHttpConfigurer<AuthorizeHttpRequestsConfigurer<H>, H> {
5959

60-
static final AuthorizationManager<RequestAuthorizationContext> permitAllAuthorizationManager = (a,
61-
o) -> new AuthorizationDecision(true);
60+
static final AuthorizationDecision AUTHORIZATION_DECISION = new AuthorizationDecision(true);
61+
62+
static final AuthorizationManager<RequestAuthorizationContext> PERMIT_ALL_AUTHORIZATION_MANAGER = (a,
63+
o) -> AUTHORIZATION_DECISION;
6264

6365
private final AuthorizationManagerRequestMatcherRegistry registry;
6466

@@ -287,7 +289,7 @@ public AuthorizedUrl not() {
287289
* customizations
288290
*/
289291
public AuthorizationManagerRequestMatcherRegistry permitAll() {
290-
return access(permitAllAuthorizationManager);
292+
return access(PERMIT_ALL_AUTHORIZATION_MANAGER);
291293
}
292294

293295
/**

config/src/main/java/org/springframework/security/config/annotation/web/configurers/PermitAllSupport.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ static void permitAll(HttpSecurityBuilder<? extends HttpSecurityBuilder<?>> http
6363
SecurityConfig.createList(ExpressionUrlAuthorizationConfigurer.permitAll)));
6464
}
6565
else {
66-
httpConfigurer.addFirst(matcher, AuthorizeHttpRequestsConfigurer.permitAllAuthorizationManager);
66+
httpConfigurer.addFirst(matcher, AuthorizeHttpRequestsConfigurer.PERMIT_ALL_AUTHORIZATION_MANAGER);
6767
}
6868
}
6969
}

0 commit comments

Comments
 (0)