Skip to content
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

Added rate limiting evaluation window #131

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ezafeire
Copy link

Added the ability to define an evaluation window when creating a rate_based_statement in a rule.

Description

While implementing rate limit policies on WAF for a client, I realized that there was no way to lower the evaluation window that WAF uses to determine rate limiting requests to under 300 sec.
With this PR you can modify it and if you don't it defaults to 300 sec (the default 5 minutes).

Added the ability to define an evaluation window when creating a rate_based_statement in a rule.
@dmzeus
Copy link

dmzeus commented Apr 15, 2024

It's a minor change but much needed)

@stavbernazport
Copy link

This is very needed!!
Took me some time to understand why it's not applying until I found out it's not yet supported by this module

Copy link
Contributor

@MatanHeledPort MatanHeledPort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🙏🏻

@stavbernazport
Copy link

@splichy Can you please check if this can be released?

@dmzeus
Copy link

dmzeus commented Sep 4, 2024

Any news here?)

@realfredlai
Copy link

Any update on this PR please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants