Merge pull request #1366 from ban-xiu/fix-issue-1204-4.3.x

Fix issue #1204

Author: ryanjbaxter

spring-cloud-openfeign-core/src/main/java/org/springframework/cloud/openfeign/encoding/FeignAcceptGzipEncodingAutoConfiguration.java

@@ -16,11 +16,9 @@
 
 package org.springframework.cloud.openfeign.encoding;
 
-import feign.Client;
 import feign.Feign;
 
 import org.springframework.boot.autoconfigure.AutoConfigureAfter;
-import org.springframework.boot.autoconfigure.condition.ConditionalOnBean;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnClass;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
 import org.springframework.boot.context.properties.EnableConfigurationProperties;
@@ -39,7 +37,6 @@
 @Configuration(proxyBeanMethods = false)
 @EnableConfigurationProperties(FeignClientEncodingProperties.class)
 @ConditionalOnClass(Feign.class)
-@ConditionalOnBean(Client.class)
 @ConditionalOnProperty("spring.cloud.openfeign.compression.response.enabled")
 // The OK HTTP client uses "transparent" compression.
 // If the accept-encoding header is present, it disables transparent compression.

spring-cloud-openfeign-core/src/test/java/org/springframework/cloud/openfeign/FeignCompressionTests.java

@@ -74,6 +74,18 @@ void shouldNotAddInterceptorsIfFeignOkHttpClientPresent() {
 			});
 	}
 
+	@Test
+	void shouldAddResponseCompressionInterceptorWithoutClientBean() {
+		new ApplicationContextRunner()
+			.withPropertyValues("spring.cloud.openfeign.compression.response.enabled=true",
+					"spring.cloud.openfeign.okhttp.enabled=false")
+			.withConfiguration(AutoConfigurations.of(FeignAutoConfiguration.class,
+					FeignAcceptGzipEncodingAutoConfiguration.class))
+			.run(context -> {
+				assertThat(context).hasSingleBean(FeignAcceptGzipEncodingInterceptor.class);
+			});
+	}
+
 	@Configuration
 	static class OkHttpClientConfiguration {