Skip to content

S
spring-cloud-netflix
Commit 8b81eb8b by Dave Syer
Options

Tweak autoconfig for feign

It doesn't make sense to @EnableFeignClients in autoconfig because the user needs to specify a package to scan. It does make sense (sort of) to set up the encoder/decoder/logger etc. See gh-226
parent 1d358048
Showing with 2 additions and 1 deletion
spring-cloud-netflix-core/src/main/java/org/springframework/cloud/netflix/feign/FeignAutoConfiguration.java
...@@ -20,6 +20,7 @@ import org.springframework.boot.autoconfigure.AutoConfigureAfter; ...@@ -20,6 +20,7 @@ import org.springframework.boot.autoconfigure.AutoConfigureAfter;
import org.springframework.boot.autoconfigure.condition.ConditionalOnClass; import org.springframework.boot.autoconfigure.condition.ConditionalOnClass;
import org.springframework.cloud.netflix.archaius.ArchaiusAutoConfiguration; import org.springframework.cloud.netflix.archaius.ArchaiusAutoConfiguration;
import org.springframework.context.annotation.Configuration; import org.springframework.context.annotation.Configuration;
import org.springframework.context.annotation.Import;
import feign.Feign; import feign.Feign;
...@@ -30,7 +31,7 @@ import feign.Feign; ...@@ -30,7 +31,7 @@ import feign.Feign;
@Configuration @Configuration
@ConditionalOnClass(Feign.class) @ConditionalOnClass(Feign.class)
@AutoConfigureAfter(ArchaiusAutoConfiguration.class) @AutoConfigureAfter(ArchaiusAutoConfiguration.class)
@EnableFeignClients @Import(FeignClientsConfiguration.class)
public class FeignAutoConfiguration { public class FeignAutoConfiguration {
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment