fizzgate/fizz-gateway-node
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Http status 4xxx not included in error flow

Browse Source
This commit is contained in:
hongqiaowei
2022-07-11 10:31:12 +08:00
parent 4beb93c359
commit 4dd9be1590
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
fizz-core/src/main/java/we/filter/FlowControlFilter.java
View File

@@ -239,7 +239,8 @@ public class FlowControlFilter extends FizzWebFilter {
if (t instanceof TimeoutException) { if (t instanceof TimeoutException) {
statusCode = HttpStatus.GATEWAY_TIMEOUT; statusCode = HttpStatus.GATEWAY_TIMEOUT;
} }
if (s == SignalType.ON_ERROR || statusCode.is4xxClientError() || statusCode.is5xxServerError()) { // if (s == SignalType.ON_ERROR || statusCode.is4xxClientError() || statusCode.is5xxServerError()) {
if (s == SignalType.ON_ERROR || statusCode.is5xxServerError()) {
flowStat.addRequestRT(resourceConfigs, currentTimeSlot, rt, false, statusCode); flowStat.addRequestRT(resourceConfigs, currentTimeSlot, rt, false, statusCode);
if (cb != null) { if (cb != null) {
cb.transit(CircuitBreaker.State.RESUME_DETECTIVE, CircuitBreaker.State.OPEN, currentTimeSlot, flowStat); cb.transit(CircuitBreaker.State.RESUME_DETECTIVE, CircuitBreaker.State.OPEN, currentTimeSlot, flowStat);