You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The name of this annotation is confusing, since it applies to the "rate limit" function only.
Would be good to rename it to rate-limit-whitelist.
In the mean time, I think the annotation whitelist-with-rate-limit is redundant and that we could apply the whitelist to the rate-limit only when rate-limit-whitelist is set.
The text was updated successfully, but these errors were encountered:
This was fixed at some point in 1.4 whitelist annotation will simply allow a set of IP/CIDR and block the rest at the L4 layer. There is no rate-limit involved.
The name of this annotation is confusing, since it applies to the "rate limit" function only.
Would be good to rename it to
rate-limit-whitelist
.In the mean time, I think the annotation
whitelist-with-rate-limit
is redundant and that we could apply the whitelist to the rate-limit only whenrate-limit-whitelist
is set.The text was updated successfully, but these errors were encountered: