fix(cachingFilter: Allow disabling the content caching filter (#1699) (…

…#1702) * fix(cachingFilter: Allow disabling the content caching filter * fix(cachingFilter: Allow disabling the content caching filter (cherry picked from commit 7e403b0) Co-authored-by: Jason <[email protected]>
spinnaker · Sep 1, 2023 · 431b73f · 431b73f
1 parent 92fa197
commit 431b73f
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/gate-web/src/main/groovy/com/netflix/spinnaker/gate/config/GateWebConfig.groovy b/gate-web/src/main/groovy/com/netflix/spinnaker/gate/config/GateWebConfig.groovy
@@ -23,6 +23,7 @@ import com.netflix.spinnaker.gate.interceptors.ResponseHeaderInterceptor
 import com.netflix.spinnaker.gate.interceptors.ResponseHeaderInterceptorConfigurationProperties
 import com.netflix.spinnaker.gate.retrofit.UpstreamBadRequest
 import com.netflix.spinnaker.kork.dynamicconfig.DynamicConfigService
+import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
 import com.netflix.spinnaker.kork.web.interceptors.MetricsInterceptor
 import org.springframework.beans.factory.annotation.Autowired
 import org.springframework.beans.factory.annotation.Value
@@ -80,7 +81,10 @@ public class GateWebConfig implements WebMvcConfigurer {
  return new HandlerMappingIntrospector(context)
  }
 
+
+ // Add the ability to disable as this breaks numerous integration patterns
  @Bean
+ @ConditionalOnProperty(value = "content.cachingFilter.enabled", matchIfMissing = true)
  Filter contentCachingFilter() {
  // This filter simply buffers the response so that Content-Length header can be set
  return new ContentCachingFilter()