Skip to content

Commit

Permalink
[SHIBUI-808]
Browse files Browse the repository at this point in the history
fix conditional configuration
  • Loading branch information
jj committed Oct 5, 2018
1 parent 5d24f3d commit a54ba7c
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
@@ -1,15 +1,14 @@
package edu.internet2.tier.shibboleth.admin.ui.configuration.auto;

import org.springframework.beans.factory.annotation.Value;
import org.springframework.boot.autoconfigure.condition.ConditionalOnClass;
import org.springframework.boot.autoconfigure.AutoConfigureOrder;
import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;
import org.springframework.context.annotation.Profile;
import org.springframework.security.config.annotation.authentication.builders.AuthenticationManagerBuilder;
import org.springframework.security.config.annotation.web.builders.HttpSecurity;
import org.springframework.security.config.annotation.web.builders.WebSecurity;
import org.springframework.security.config.annotation.web.configuration.EnableWebSecurity;
import org.springframework.security.config.annotation.web.configuration.WebSecurityConfigurerAdapter;
import org.springframework.security.web.csrf.CookieCsrfTokenRepository;
import org.springframework.security.web.firewall.HttpFirewall;
Expand All @@ -22,6 +21,8 @@
* Workaround for slashes in URL from [https://stackoverflow.com/questions/48453980/spring-5-0-3-requestrejectedexception-the-request-was-rejected-because-the-url]
*/
@Configuration
@AutoConfigureOrder(Integer.MAX_VALUE - 100)
@ConditionalOnMissingBean(WebSecurityConfigurerAdapter.class)
public class WebSecurityConfig {

@Value("${shibui.logout-url:/dashboard}")
Expand All @@ -38,7 +39,6 @@ public HttpFirewall allowUrlEncodedSlashHttpFirewall() {
}

@Bean
@ConditionalOnMissingBean(name = "webSecurityConfig")
@Profile("!no-auth")
public WebSecurityConfigurerAdapter defaultAuth() {
return new WebSecurityConfigurerAdapter() {
Expand Down

0 comments on commit a54ba7c

Please sign in to comment.