diff --git a/config/src/main/java/org/springframework/security/config/web/server/ServerHttpSecurity.java b/config/src/main/java/org/springframework/security/config/web/server/ServerHttpSecurity.java index da40711f12..97b5e72840 100644 --- a/config/src/main/java/org/springframework/security/config/web/server/ServerHttpSecurity.java +++ b/config/src/main/java/org/springframework/security/config/web/server/ServerHttpSecurity.java @@ -543,17 +543,17 @@ public class ServerHttpSecurity { private ServerAuthenticationFailureHandler authenticationFailureHandler; - private ServerAuthenticationSuccessHandler serverAuthenticationSuccessHandler = this.defaultSuccessHandler; + private ServerAuthenticationSuccessHandler authenticationSuccessHandler = this.defaultSuccessHandler; public FormLoginSpec authenticationManager(ReactiveAuthenticationManager authenticationManager) { this.authenticationManager = authenticationManager; return this; } - public FormLoginSpec serverAuthenticationSuccessHandler( - ServerAuthenticationSuccessHandler serverAuthenticationSuccessHandler) { - Assert.notNull(serverAuthenticationSuccessHandler, "serverAuthenticationSuccessHandler cannot be null"); - this.serverAuthenticationSuccessHandler = serverAuthenticationSuccessHandler; + public FormLoginSpec authenticationSuccessHandler( + ServerAuthenticationSuccessHandler authenticationSuccessHandler) { + Assert.notNull(authenticationSuccessHandler, "authenticationSuccessHandler cannot be null"); + this.authenticationSuccessHandler = authenticationSuccessHandler; return this; } @@ -614,7 +614,7 @@ public class ServerHttpSecurity { authenticationFilter.setRequiresAuthenticationMatcher(this.requiresAuthenticationMatcher); authenticationFilter.setAuthenticationFailureHandler(this.authenticationFailureHandler); authenticationFilter.setAuthenticationConverter(new ServerFormLoginAuthenticationConverter()); - authenticationFilter.setServerAuthenticationSuccessHandler(this.serverAuthenticationSuccessHandler); + authenticationFilter.setAuthenticationSuccessHandler(this.authenticationSuccessHandler); authenticationFilter.setSecurityContextRepository(this.securityContextRepository); http.addFilterAt(authenticationFilter, SecurityWebFiltersOrder.FORM_LOGIN); } diff --git a/config/src/test/java/org/springframework/security/config/web/server/FormLoginTests.java b/config/src/test/java/org/springframework/security/config/web/server/FormLoginTests.java index 921bdaf921..3dc4ba41b7 100644 --- a/config/src/test/java/org/springframework/security/config/web/server/FormLoginTests.java +++ b/config/src/test/java/org/springframework/security/config/web/server/FormLoginTests.java @@ -124,7 +124,7 @@ public class FormLoginTests { .anyExchange().authenticated() .and() .formLogin() - .serverAuthenticationSuccessHandler(new RedirectServerAuthenticationSuccessHandler("/custom")) + .authenticationSuccessHandler(new RedirectServerAuthenticationSuccessHandler("/custom")) .and() .build(); diff --git a/web/src/main/java/org/springframework/security/web/server/authentication/AuthenticationWebFilter.java b/web/src/main/java/org/springframework/security/web/server/authentication/AuthenticationWebFilter.java index 09a2849981..da3096b45f 100644 --- a/web/src/main/java/org/springframework/security/web/server/authentication/AuthenticationWebFilter.java +++ b/web/src/main/java/org/springframework/security/web/server/authentication/AuthenticationWebFilter.java @@ -43,7 +43,7 @@ public class AuthenticationWebFilter implements WebFilter { private final ReactiveAuthenticationManager authenticationManager; - private ServerAuthenticationSuccessHandler serverAuthenticationSuccessHandler = new WebFilterChainServerAuthenticationSuccessHandler(); + private ServerAuthenticationSuccessHandler authenticationSuccessHandler = new WebFilterChainServerAuthenticationSuccessHandler(); private Function> authenticationConverter = new ServerHttpBasicAuthenticationConverter(); @@ -81,7 +81,7 @@ public class AuthenticationWebFilter implements WebFilter { SecurityContextImpl securityContext = new SecurityContextImpl(); securityContext.setAuthentication(authentication); return this.securityContextRepository.save(exchange, securityContext) - .then(this.serverAuthenticationSuccessHandler + .then(this.authenticationSuccessHandler .onAuthenticationSuccess(webFilterExchange, authentication)); } @@ -91,8 +91,8 @@ public class AuthenticationWebFilter implements WebFilter { this.securityContextRepository = securityContextRepository; } - public void setServerAuthenticationSuccessHandler(ServerAuthenticationSuccessHandler serverAuthenticationSuccessHandler) { - this.serverAuthenticationSuccessHandler = serverAuthenticationSuccessHandler; + public void setAuthenticationSuccessHandler(ServerAuthenticationSuccessHandler authenticationSuccessHandler) { + this.authenticationSuccessHandler = authenticationSuccessHandler; } public void setAuthenticationConverter(Function> authenticationConverter) { diff --git a/web/src/test/java/org/springframework/security/web/server/authentication/AuthenticationWebFilterTests.java b/web/src/test/java/org/springframework/security/web/server/authentication/AuthenticationWebFilterTests.java index 48ddf46d87..745cfc9aa4 100644 --- a/web/src/test/java/org/springframework/security/web/server/authentication/AuthenticationWebFilterTests.java +++ b/web/src/test/java/org/springframework/security/web/server/authentication/AuthenticationWebFilterTests.java @@ -69,7 +69,7 @@ public class AuthenticationWebFilterTests { @Before public void setup() { this.filter = new AuthenticationWebFilter(this.authenticationManager); - this.filter.setServerAuthenticationSuccessHandler(this.successHandler); + this.filter.setAuthenticationSuccessHandler(this.successHandler); this.filter.setAuthenticationConverter(this.authenticationConverter); this.filter.setSecurityContextRepository(this.securityContextRepository); this.filter.setAuthenticationFailureHandler(this.failureHandler);