[rzukow]

What do you think about adding some ConditionalOnMissingBean annotation to pre filters?
The problem we are facing is DebugFilter that is calling request.getParameter(DEBUG_PARAMETER.get()) just to check the value of debug and ignore it, and there is no easy way to disable it.

I added some other beans as described here: https://www.anaplan.com/blog/using-zuul-in-production/ , but the article is outdated anyway.

[spencergibb]

Please remove the name parameter

[codecov]

Codecov Report

Merging #3609 into master will increase coverage by 2.83%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master   #3609      +/-   ##
===========================================
+ Coverage     62.16%     65%   +2.83%     
- Complexity      667    1504     +837     
===========================================
  Files            98     190      +92     
  Lines          3124    7272    +4148     
  Branches        342     870     +528     
===========================================
+ Hits           1942    4727    +2785     
- Misses         1060    2235    +1175     
- Partials        122     310     +188

Impacted Files	Coverage Δ	Complexity Δ
...loud/netflix/zuul/ZuulServerAutoConfiguration.java	70.31% <ø> (ø)	18 <0> (?)
...k/cloud/netflix/eureka/EurekaClientConfigBean.java	28.07% <0%> (-0.64%)	45% <0%> (-2%)
...ud/netflix/zuul/ZuulServerMarkerConfiguration.java	100% <0%> (ø)	2% <0%> (?)
...etflix/turbine/stream/TurbineStreamProperties.java	0% <0%> (ø)	0% <0%> (?)
...turbine/stream/TurbineStreamAutoConfiguration.java	0% <0%> (ø)	0% <0%> (?)
...ul/filters/discovery/SimpleServiceRouteMapper.java	100% <0%> (ø)	2% <0%> (?)
...d/netflix/eureka/server/EurekaServerBootstrap.java	55.38% <0%> (ø)	6% <0%> (?)
...ix/sidecar/LocalApplicationHealthCheckHandler.java	100% <0%> (ø)	5% <0%> (?)
.../netflix/zuul/filters/route/SendForwardFilter.java	77.77% <0%> (ø)	7% <0%> (?)
...ework/cloud/netflix/sidecar/SidecarController.java	16.66% <0%> (ø)	1% <0%> (?)
... and 84 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d40b5d...7b04f68. Read the comment docs.

[rzukow]

Please remove the name parameter

Done.