-
Notifications
You must be signed in to change notification settings - Fork 361
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
Add rateLimit to BackendTrafficPolicy #2060
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2060 +/- ##
==========================================
- Coverage 65.17% 65.06% -0.12%
==========================================
Files 99 99
Lines 14410 14536 +126
==========================================
+ Hits 9392 9458 +66
- Misses 4437 4491 +54
- Partials 581 587 +6
|
What happens if both a "RateLimitFilter" and the "RateLimit" of a "BackendTrafficPolicy" attach the same route? Will "RateLimitFilter" API be deprecated? |
@zhaohuabing the plan is to remove RateLimitFilter (& AuthenticationFilter) before we release v0.6 |
Relates to envoyproxy#1821 & envoyproxy#2006 Signed-off-by: Arko Dasgupta <arko@tetrate.io>
43640a4
to
d281a7a
Compare
Relates to #1821 & #2006