-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Circuit breaker improvement #2660
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
t.Run("Extended Paths", func(t *testing.T) { | ||
BuildAndLoadAPI(func(spec *APISpec) { | ||
UpdateAPIVersion(spec, "v1", func(v *apidef.VersionInfo) { | ||
json.Unmarshal([]byte(`[ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feels like formatting error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
Apart from small formatting issue looks goood |
23ddc95
to
feaf7af
Compare
/release to release-2.9 |
Working on it! Note that it can take a few minutes. |
Still working... |
Tyk's Circuit Breaker only works for 500 errors. This is pretty limiting. e.g. the breaker will not trip for failed upstreams. This PR fixes that in order to break the circuit for any 5XX errors or if RoundTrip returns an error. (cherry picked from commit f6d25e6)
@furkansenharputlu Succesfully merged |
Tyk's Circuit Breaker only works for 500 errors. This is pretty limiting. e.g. the breaker will not trip for failed upstreams.
This PR fixes that in order to break the circuit for any 5XX errors or if RoundTrip returns an error.