Skip to content

Commit

Permalink
fix(ldap): Disable the broken LdapHealthIndicatorAutoConfiguration
Browse files Browse the repository at this point in the history
We can reenable it once we upgrade to a version of spring-boot with the
fix for spring-projects/spring-ldap#473
  • Loading branch information
plumpy committed May 10, 2019
1 parent 833edc2 commit 01759aa
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
package com.netflix.spinnaker.gate

import com.netflix.spinnaker.hystrix.spectator.HystrixSpectatorConfig
import org.springframework.boot.actuate.autoconfigure.ldap.LdapHealthIndicatorAutoConfiguration
import org.springframework.boot.autoconfigure.EnableAutoConfiguration
import org.springframework.boot.autoconfigure.groovy.template.GroovyTemplateAutoConfiguration
import org.springframework.boot.autoconfigure.gson.GsonAutoConfiguration
Expand All @@ -32,7 +33,7 @@ import org.springframework.scheduling.annotation.EnableAsync
@EnableConfigurationProperties
@Import([HystrixSpectatorConfig])
@ComponentScan(["com.netflix.spinnaker.gate", "com.netflix.spinnaker.config"])
@EnableAutoConfiguration(exclude = [GroovyTemplateAutoConfiguration, GsonAutoConfiguration])
@EnableAutoConfiguration(exclude = [GroovyTemplateAutoConfiguration, GsonAutoConfiguration, LdapHealthIndicatorAutoConfiguration])
class Main {

static final Map<String, String> DEFAULT_PROPS = [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ import com.netflix.spinnaker.kork.dynamicconfig.DynamicConfigService
import com.netflix.spinnaker.kork.dynamicconfig.SpringDynamicConfigService
import com.netflix.spinnaker.kork.web.exceptions.GenericExceptionHandlers
import org.springframework.boot.SpringApplication
import org.springframework.boot.actuate.autoconfigure.ldap.LdapHealthIndicatorAutoConfiguration
import org.springframework.boot.autoconfigure.EnableAutoConfiguration
import org.springframework.boot.autoconfigure.groovy.template.GroovyTemplateAutoConfiguration
import org.springframework.boot.autoconfigure.gson.GsonAutoConfiguration
Expand Down Expand Up @@ -253,7 +254,7 @@ class FunctionalSpec extends Specification {
}
@Order(10)
@EnableAutoConfiguration(exclude = [GroovyTemplateAutoConfiguration, GsonAutoConfiguration])
@EnableAutoConfiguration(exclude = [GroovyTemplateAutoConfiguration, GsonAutoConfiguration, LdapHealthIndicatorAutoConfiguration])
private static class FunctionalConfiguration extends WebSecurityConfigurerAdapter {
@Bean
Expand Down

0 comments on commit 01759aa

Please sign in to comment.