Replies: 2 comments
-
Hi @shaun-nx thanks for reporting! Be sure to check out the docs and the Contributing Guidelines while you wait for a human to take a look at this 🙂 Cheers! |
Beta Was this translation helpful? Give feedback.
0 replies
-
@shaun-nx todo: add comments |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Overview
Hi folks!
This discussion was originally posted as a bug, but this has been converted to a discussion to allow this use case to be openly discussed.
The use case here is, ensuring that teach route that references the same rateLimit Policy has their own Rate Limit Request Zone.
In the current implementation, a Rate Limit Request Zone is created for each Policy deployed, rater than for each reference to that policy.
Steps to reproduce
Actual Behaviour
I have a VirtualServer with 3 routes:
/coffee
with a basic rate-limit policy applied/tea
with a basic rate-limit policy applied/global-rate
which has no policy applied, as I want any number of requests to hit this endpointWith this setup, if
/coffee
is being hit by more than 1 request per-second, any request to/tea
will return a 503.In order to get around this, I would need to create a separate, identical policy with the same rate limit settings.
This is because the rate limit zones is tied to the Policy itself and not the location that it is supplied to.
Here is an example of the YAML files I am using:
Policy:
VirtualServer:
The below command will send a request and sleep for 1 second. This is to simulate the behaviour
If a request is sent to
/tea
while this loop is running, the endpoint will return503
If I hit the
/unlimited
endpoint, it will work fine since there is no policy attached:Expected behaviour
I would expect
/coffee
and/tea
to both have independent rate limiting.As it is, if one endpoint is overwhelemd, all other endpoints using the same policy will be unavailable
Beta Was this translation helpful? Give feedback.
All reactions