From 5d7f11bbbd3a849342251e0311e5c88efebbd9bb Mon Sep 17 00:00:00 2001 From: plll0123 Date: Mon, 10 Feb 2025 04:48:50 +0900 Subject: [PATCH] Refactor authorization manager variable naming - Renamed PERMIT_ALL_AUTHORIZATION_MANAGER to snake_case style - Introduced AUTHORIZATION_DECISION for reuse Signed-off-by: plll0123 --- .../web/configurers/AuthorizeHttpRequestsConfigurer.java | 8 +++++--- .../annotation/web/configurers/PermitAllSupport.java | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/config/src/main/java/org/springframework/security/config/annotation/web/configurers/AuthorizeHttpRequestsConfigurer.java b/config/src/main/java/org/springframework/security/config/annotation/web/configurers/AuthorizeHttpRequestsConfigurer.java index d3f76a3f51e..79a5d83af5e 100644 --- a/config/src/main/java/org/springframework/security/config/annotation/web/configurers/AuthorizeHttpRequestsConfigurer.java +++ b/config/src/main/java/org/springframework/security/config/annotation/web/configurers/AuthorizeHttpRequestsConfigurer.java @@ -57,8 +57,10 @@ public final class AuthorizeHttpRequestsConfigurer> extends AbstractHttpConfigurer, H> { - static final AuthorizationManager permitAllAuthorizationManager = (a, - o) -> new AuthorizationDecision(true); + static final AuthorizationDecision AUTHORIZATION_DECISION = new AuthorizationDecision(true); + + static final AuthorizationManager PERMIT_ALL_AUTHORIZATION_MANAGER = (a, + o) -> AUTHORIZATION_DECISION; private final AuthorizationManagerRequestMatcherRegistry registry; @@ -287,7 +289,7 @@ public AuthorizedUrl not() { * customizations */ public AuthorizationManagerRequestMatcherRegistry permitAll() { - return access(permitAllAuthorizationManager); + return access(PERMIT_ALL_AUTHORIZATION_MANAGER); } /** diff --git a/config/src/main/java/org/springframework/security/config/annotation/web/configurers/PermitAllSupport.java b/config/src/main/java/org/springframework/security/config/annotation/web/configurers/PermitAllSupport.java index 83a69e7cef5..5c8028567a3 100644 --- a/config/src/main/java/org/springframework/security/config/annotation/web/configurers/PermitAllSupport.java +++ b/config/src/main/java/org/springframework/security/config/annotation/web/configurers/PermitAllSupport.java @@ -63,7 +63,7 @@ static void permitAll(HttpSecurityBuilder> http SecurityConfig.createList(ExpressionUrlAuthorizationConfigurer.permitAll))); } else { - httpConfigurer.addFirst(matcher, AuthorizeHttpRequestsConfigurer.permitAllAuthorizationManager); + httpConfigurer.addFirst(matcher, AuthorizeHttpRequestsConfigurer.PERMIT_ALL_AUTHORIZATION_MANAGER); } } }