From d78833dd9e73125e256c94d82c14562696a8738e Mon Sep 17 00:00:00 2001
From: maryarm <maryam@nilin.co>
Date: Sat, 14 Dec 2019 19:26:34 +0330
Subject: [PATCH] BAEL-3338: A Guide to AuthenticationManagerResolver in Spring
 Security -Fix issues according to review

---
 .../reactive/authresolver/CustomWebSecurityConfig.java   | 3 ++-
 .../authresolver/CustomWebSecurityConfigurer.java        | 9 +++++----
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/spring-5-reactive-security/src/main/java/com/baeldung/reactive/authresolver/CustomWebSecurityConfig.java b/spring-5-reactive-security/src/main/java/com/baeldung/reactive/authresolver/CustomWebSecurityConfig.java
index cd5a0af5a2..d07a991089 100644
--- a/spring-5-reactive-security/src/main/java/com/baeldung/reactive/authresolver/CustomWebSecurityConfig.java
+++ b/spring-5-reactive-security/src/main/java/com/baeldung/reactive/authresolver/CustomWebSecurityConfig.java
@@ -44,8 +44,9 @@ public class CustomWebSecurityConfig {
               .getPath()
               .subPath(0)
               .value()
-              .startsWith("/employee"))
+              .startsWith("/employee")) {
                 return Mono.just(employeesAuthenticationManager());
+            }
             return Mono.just(customersAuthenticationManager());
         };
     }
diff --git a/spring-5-security/src/main/java/com/baeldung/authresolver/CustomWebSecurityConfigurer.java b/spring-5-security/src/main/java/com/baeldung/authresolver/CustomWebSecurityConfigurer.java
index ec168e2897..33ef692173 100644
--- a/spring-5-security/src/main/java/com/baeldung/authresolver/CustomWebSecurityConfigurer.java
+++ b/spring-5-security/src/main/java/com/baeldung/authresolver/CustomWebSecurityConfigurer.java
@@ -27,8 +27,9 @@ public class CustomWebSecurityConfigurer extends WebSecurityConfigurerAdapter {
         return request -> {
             if (request
               .getPathInfo()
-              .startsWith("/employee"))
+              .startsWith("/employee")) {
                 return employeesAuthenticationManager();
+            }
             return customersAuthenticationManager();
         };
     }
@@ -63,10 +64,10 @@ public class CustomWebSecurityConfigurer extends WebSecurityConfigurerAdapter {
     }
 
     private AuthenticationFilter authenticationFilter() {
-        AuthenticationFilter ret = new AuthenticationFilter(
+        AuthenticationFilter filter = new AuthenticationFilter(
           resolver(), authenticationConverter());
-        ret.setSuccessHandler((request, response, auth) -> {});
-        return ret;
+        filter.setSuccessHandler((request, response, auth) -> {});
+        return filter;
     }
 
     private AuthenticationManager employeesAuthenticationManager() {