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.

Sign up for GitHub

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

Jump to bottom

4.6.1 #533

Merged
merged 6 commits into from
Feb 4, 2025
Merged

4.6.1 #533

merged 6 commits into from
Feb 4, 2025

Conversation

daknhh
Copy link
Contributor

@daknhh daknhh commented Feb 4, 2025

Fixed

  • Bug Fix: Resolved "code not found" error in WAF WCU Quota check. Kudos to @vboufleur for fixing this.
  • WAFConfig Updates: Kudos to @vboufleur for fixing this.
    • Added OptimizeUnassociatedWebACL property.
    • Fixed an issue where CDK rejected deployments if the first character of CustomResponseBodies properties was uppercase.
  • Fix: Corrected imports in UnusedNotification Lambda.

vboufleur and others added 6 commits January 31, 2025 18:18
@vboufleur
Fix "not found" error on WAF WCU Quota check
ded4e4d
@vboufleur
WafConfig: add prop + must lowercase others
3e2ef19 
CDK was rejecting the deployment of the stack if the firt char of the CustomResponseBodies
props aren't lowercased. After this commit my CDK deployment worked perfectly.
@vboufleur
Bump version
972e6ec
@daknhh
Fix imports in UnusedResourceNotification
d21a2ad
@daknhh
update changelog
f95510a
@daknhh
fix workflows
2f290b9
@daknhh daknhh merged commit 2d435dc into master Feb 4, 2025
9 checks passed
@daknhh daknhh deleted the pullrequests/vboufleur/fix/misc branch February 4, 2025 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@daknhh @vboufleur