feat: support rate limit based on path and method#7046
feat: support rate limit based on path and method#7046arkodg merged 23 commits intoenvoyproxy:mainfrom
Conversation
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
e3d8030 to
fd3d99c
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #7046 +/- ##
==========================================
+ Coverage 72.03% 72.06% +0.03%
==========================================
Files 231 231
Lines 33468 33634 +166
==========================================
+ Hits 24108 24239 +131
- Misses 7604 7646 +42
+ Partials 1756 1749 -7 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
7a48c92 to
c179c5e
Compare
|
/retest |
Signed-off-by: kkk777-7 <[email protected]>
c179c5e to
28f0adc
Compare
Signed-off-by: kkk777-7 <[email protected]>
22a2a47 to
929053f
Compare
Signed-off-by: kkk777-7 <[email protected]>
internal/gatewayapi/testdata/backendtrafficpolicy-with-ratelimit.out.yaml
Show resolved
Hide resolved
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
|
@kkk777-7 can you rebase and fix the release notes conflict? |
Signed-off-by: kkk777-7 <[email protected]>
Signed-off-by: kkk777-7 <[email protected]>
f5b4fad to
d66a4b4
Compare
|
It seems that the failure is occurring when referencing Gateway API docs. |
it's a known issue, I'm working on that. #7370 will ignore it. |
|
@kkk777-7 there seems to be one issue - path prefix of |
What this PR does / why we need it:
support rate limit based on path and method.
Which issue(s) this PR fixes:
Fixes #4449
Release Notes: Yes