-
Notifications
You must be signed in to change notification settings - Fork 350
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
docs: add docs for ratelimit invert match headers #4483
docs: add docs for ratelimit invert match headers #4483
Conversation
b34764d
to
b19b8ab
Compare
Added docs manually but |
you should update docs under docs alias to current version, which means |
01f9503
to
4056304
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4483 +/- ##
=======================================
Coverage 65.97% 65.98%
=======================================
Files 203 203
Lines 31154 31154
=======================================
+ Hits 20554 20556 +2
+ Misses 9415 9414 -1
+ Partials 1185 1184 -1 ☔ View full report in Codecov by Sentry. |
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.
LGTM, the example could be improved but it's not blocking.
4056304
to
c286335
Compare
Signed-off-by: Rudrakh Panigrahi <[email protected]>
c286335
to
08352e0
Compare
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.
LGTM thanks!
Signed-off-by: Rudrakh Panigrahi <[email protected]>
8c49b64
to
2aacb82
Compare
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.
LGTM
/retest |
What type of PR is this?
docs: add docs for ratelimit invert match headers
api: remove
notImplementedHide
forInvert
featuree2e: add a test for invert header match in local ratelimit case
What this PR does / why we need it:
Docs for feature implemented in #4286
Which issue(s) this PR fixes:
Fixes #2193