From 4dd9be15908a6ed51a48a31b65730eda7fbccd5f Mon Sep 17 00:00:00 2001 From: hongqiaowei Date: Mon, 11 Jul 2022 10:31:12 +0800 Subject: [PATCH] Http status 4xxx not included in error flow --- fizz-core/src/main/java/we/filter/FlowControlFilter.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/fizz-core/src/main/java/we/filter/FlowControlFilter.java b/fizz-core/src/main/java/we/filter/FlowControlFilter.java index 34a1d34..a666412 100644 --- a/fizz-core/src/main/java/we/filter/FlowControlFilter.java +++ b/fizz-core/src/main/java/we/filter/FlowControlFilter.java @@ -239,7 +239,8 @@ public class FlowControlFilter extends FizzWebFilter { if (t instanceof TimeoutException) { 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); if (cb != null) { cb.transit(CircuitBreaker.State.RESUME_DETECTIVE, CircuitBreaker.State.OPEN, currentTimeSlot, flowStat);