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

Add dynamic rule_group_reference_statement block to attach custom rule groups #70

Conversation

surendhar-kodaikal-palani
Copy link
Contributor

@surendhar-kodaikal-palani surendhar-kodaikal-palani commented Nov 1, 2022

Description

Add dynamic rule_group_reference_statement block to attach custom rule groups which is not supported by the current module.

How was it tested ?

Ran validate, plan, and apply

      + rule {
          + name     = "rule-custom-api-allow"
          + priority = 7
          + override_action {
              + none {}
            }
          + statement {
              + rule_group_reference_statement {
                  + arn = "arn:aws:wafv2:us-west-2:******:regional/rulegroup/*******"
                }
            }
          + visibility_config {
              + cloudwatch_metrics_enabled = true
              + metric_name                = "rule-custom-api-allow"
              + sampled_requests_enabled   = true
            }
        }

@surendhar-kodaikal-palani surendhar-kodaikal-palani force-pushed the enable-rule-group-reference-statement branch 3 times, most recently from 22c17d5 to d8a9d70 Compare November 2, 2022 02:34
@lberbel
Copy link

lberbel commented Jan 23, 2023

I needed it as well!

@Ohid25 Ohid25 force-pushed the enable-rule-group-reference-statement branch from f6eb39e to fdb7d16 Compare January 25, 2023 20:07
@Ohid25 Ohid25 merged commit ee48d46 into umotif-public:main Jan 25, 2023
@Ohid25
Copy link
Contributor

Ohid25 commented Jan 26, 2023

Part of the 4.0.0 release! Thank you for your contribution!

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.

3 participants