Skip to content

Commit fa3a6d4

Browse files
committed
Merge pull request 'Fix #24152: make sure that authenticationManagerBean is available for SAML and Kerberos' (#26) from bugfix/24152 into develop
2 parents 6640463 + 106afff commit fa3a6d4

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

src/main/java/eu/openanalytics/containerproxy/security/WebSecurityConfig.java

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,13 @@
2525
import javax.inject.Inject;
2626

2727
import org.springframework.beans.factory.annotation.Autowired;
28+
import org.springframework.boot.autoconfigure.condition.ConditionalOnExpression;
29+
import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
2830
import org.springframework.context.annotation.Bean;
2931
import org.springframework.context.annotation.Configuration;
3032
import org.springframework.core.env.Environment;
3133
import org.springframework.security.authentication.AuthenticationEventPublisher;
34+
import org.springframework.security.authentication.AuthenticationManager;
3235
import org.springframework.security.config.annotation.authentication.builders.AuthenticationManagerBuilder;
3336
import org.springframework.security.config.annotation.authentication.configuration.GlobalAuthenticationConfigurerAdapter;
3437
import org.springframework.security.config.annotation.web.builders.HttpSecurity;
@@ -162,5 +165,12 @@ public void init(AuthenticationManagerBuilder amb) throws Exception {
162165
}
163166
};
164167
}
165-
168+
169+
@Bean(name="authenticationManager")
170+
@ConditionalOnExpression("'${proxy.authentication}' == 'kerberos' || '${proxy.authentication}' == 'saml'")
171+
@Override
172+
public AuthenticationManager authenticationManagerBean() throws Exception {
173+
return super.authenticationManagerBean();
174+
}
175+
166176
}

0 commit comments

Comments
 (0)