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

Higher limit lint value for AWS WAF2 Ratelimit #3738

Open
vaclavbarta opened this issue Oct 9, 2024 · 5 comments
Open

Higher limit lint value for AWS WAF2 Ratelimit #3738

vaclavbarta opened this issue Oct 9, 2024 · 5 comments
Labels
spec CloudFormation Specification Bug

Comments

@vaclavbarta
Copy link

Hello,
sorry - I'm not sure if this repository relates to my issues, but i think so.

AWS announced at august 2024 the lowering the limit for WAF RateLimit's limit from 100 to 10 here https://aws.amazon.com/about-aws/whats-new/2024/08/aws-waf-rate-based-rules-lower-rate-limits/ but this value cannot be used due cloudformation validation.

But i can't use it through cloudformation (cdk) because values lower than 100 failing on Cloudformation error

Resource handler returned message: "Model validation failed (#/Rules: 2 schema violations found) #/Rules/8/Statement/RateBasedStatement/Limit: failed validation constraint for keyword [minimum] (#/Rules/8/Statement/RateBasedStatement/Limit) #/Rules/9/Statement/RateBasedStatement/Limit: failed validation constraint for keyword [minimum] (#/Rules/9/Statement/RateBasedStatement/Limit)" (RequestToken: REDACTED-606b89, HandlerErrorCode: InvalidRequest)

I found that it is defined also here. Should it be synced please ? :)

"RateLimit": {
"maximum": 2000000000,
"minimum": 100,
"type": "integer"
},

Thank you very much Vaclav

@kddejong kddejong added the spec CloudFormation Specification Bug label Oct 9, 2024
@kddejong
Copy link
Contributor

kddejong commented Oct 9, 2024

So we get our resource schemas from docs. Unfortunately it is wrong there and that is what service side validation uses. So while we can change it you will still get there.

@vaclavbarta
Copy link
Author

Thank you @kddejong for fast response. I checked the zip and really there is 100 🤔 - Didn't you know where this can be reported to somebody who can fix it ? 😎

@vaclavbarta
Copy link
Author

I try wrote it to Twitter so we see ;) https://x.com/AWSSupport/status/1844069569633161571

@kddejong
Copy link
Contributor

kddejong commented Oct 9, 2024

I can see that its in the process of being changed but I can't say when it will be complete. Hopefully it will be soon.

@vaclavbarta
Copy link
Author

It seems already changed 😉 we will test it and i will close this issue soon

https://github.com/aws-cloudformation/cfn-lint/blob/main/src/cfnlint/data/schemas/providers/us_east_1/aws-wafv2-rulegroup.json#L771

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

No branches or pull requests

2 participants