OP-22248: setting auth manager to SecurityFilterChain to directly to …

…ensure right object is set.
OpsMx · Jun 14, 2024 · e992eb8 · e992eb8
1 parent bfec5b8
commit e992eb8
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/gate-basic/src/main/java/com/netflix/spinnaker/gate/security/basic/BasicAuthConfig.java b/gate-basic/src/main/java/com/netflix/spinnaker/gate/security/basic/BasicAuthConfig.java
@@ -65,8 +65,7 @@ public BasicAuthConfig(AuthConfig authConfig, BasicAuthProvider authProvider) {
     this.authProvider = authProvider;
   }
 
-  @Bean
-  public AuthenticationManager authManager(HttpSecurity http) throws Exception {
+  private AuthenticationManager authManager(HttpSecurity http) throws Exception {
     AuthenticationManagerBuilder authenticationManagerBuilder =
         http.getSharedObject(AuthenticationManagerBuilder.class);
     if (name == null || name.isEmpty() || password == null || password.isEmpty()) {
@@ -95,7 +94,7 @@ public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
     http.csrf().disable();
     http.formLogin()
         .and()
-        //        .authenticationManager(authManager(http))
+        .authenticationManager(authManager(http))
         .httpBasic()
         .authenticationEntryPoint(new LoginUrlAuthenticationEntryPoint("/login"));
     authConfig.configure(http);