Skip to content

Add resumeWithoutException to CircuitBreakerFilter#2082

Merged
ryanjbaxter merged 1 commit intospring-cloud:mainfrom
dlsrb6342:issue-2047
Jun 1, 2021
Merged

Add resumeWithoutException to CircuitBreakerFilter#2082
ryanjbaxter merged 1 commit intospring-cloud:mainfrom
dlsrb6342:issue-2047

Conversation

@dlsrb6342
Copy link
Contributor

@codecov
Copy link

codecov bot commented Dec 17, 2020

Codecov Report

Merging #2082 (1390eea) into master (d2ea0eb) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2082   +/-   ##
=========================================
  Coverage     77.16%   77.16%           
- Complexity     1274     1276    +2     
=========================================
  Files           164      164           
  Lines          4895     4901    +6     
  Branches        402      403    +1     
=========================================
+ Hits           3777     3782    +5     
- Misses          909      910    +1     
  Partials        209      209           
Impacted Files Coverage Δ Complexity Δ
...actory/SpringCloudCircuitBreakerFilterFactory.java 92.30% <100.00%> (+0.50%) 9.00 <0.00> (ø)
...gCloudCircuitBreakerResilience4JFilterFactory.java 100.00% <100.00%> (ø) 5.00 <0.00> (+1.00)
...cloud/gateway/filter/NettyWriteResponseFilter.java 78.26% <0.00%> (-4.35%) 16.00% <0.00%> (-1.00%)
...eway/filter/factory/RetryGatewayFilterFactory.java 77.97% <0.00%> (+0.59%) 38.00% <0.00%> (+2.00%)

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 d2ea0eb...1390eea. Read the comment docs.

@dlsrb6342
Copy link
Contributor Author

Hi @spencergibb
Could you have a look this PR?

@ryanjbaxter ryanjbaxter merged commit 393515c into spring-cloud:main Jun 1, 2021
@dlsrb6342 dlsrb6342 deleted the issue-2047 branch June 8, 2021 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Response body can be modified because of SpringCloudCircuitBreakerFilterFactory

4 participants