Skip to content

Commit

Permalink
fix(fiat): Set @EnableFiatAutoConfig to avoid unnecessary auth requ…
Browse files Browse the repository at this point in the history
…irements (#286)

Original intention was to not require full blown `fiat` to use
`FiatStatus` but unfortunately spring security is brought in as a
transitive dependency.

This PR is a simpler alternative to moving `FiatStatus` and the
related configuration properties classes to a new artifact that
would not bring in any unnecessary dependencies.

fixes spinnaker/spinnaker#2949
  • Loading branch information
ajordens authored Jun 25, 2018
1 parent 0c19f49 commit 8229c75
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ package com.netflix.spinnaker.echo.config
import com.google.common.collect.ImmutableList
import com.netflix.appinfo.InstanceInfo
import com.netflix.spectator.api.Registry
import com.netflix.spinnaker.fiat.shared.EnableFiatAutoConfig
import com.netflix.spinnaker.kork.web.interceptors.MetricsInterceptor
import org.springframework.beans.factory.annotation.Autowired
import org.springframework.context.annotation.Bean
Expand All @@ -34,6 +35,7 @@ import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter
* I generally exclude @Configuration's from this scan, as picking those up can affect your tests.
*/
@Configuration
@EnableFiatAutoConfig
@ComponentScan(basePackages = ['com.netflix.spinnaker.echo'],
excludeFilters = @ComponentScan.Filter(value = Configuration,
type = FilterType.ANNOTATION))
Expand Down

0 comments on commit 8229c75

Please sign in to comment.