Skip to content

Commit

Permalink
Merge pull request #5 from dms1981/fix/minor-fixes
Browse files Browse the repository at this point in the history
Minor fixes
  • Loading branch information
dms1981 authored Nov 18, 2022
2 parents b381f24 + 311194d commit d42fcf5
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 7 deletions.
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ module "fw-rules" {
|------|---------|
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.0 |
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | ~> 4.0 |
| <a name="requirement_random"></a> [random](#requirement\_random) | ~> 3.4 |

## Providers

Expand Down
4 changes: 2 additions & 2 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ resource "aws_networkfirewall_firewall_policy" "main" {
name = replace(title(var.fw_policy_name), "/-|_/", "")
firewall_policy {
stateful_engine_options {
rule_order = "STRICT_ORDER"
rule_order = var.fw_policy_rule_order
}
stateful_rule_group_reference {
priority = 1
Expand All @@ -22,7 +22,7 @@ resource "aws_networkfirewall_rule_group" "stateful" {

rule_group {
stateful_rule_options {
rule_order = "STRICT_ORDER"
rule_order = var.fw_policy_rule_order
}
rules_source {
dynamic "stateful_rule" {
Expand Down
4 changes: 0 additions & 4 deletions versions.tf
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,6 @@ terraform {
version = "~> 4.0"
source = "hashicorp/aws"
}
random = {
source = "hashicorp/random"
version = "~> 3.4"
}
}
required_version = ">= 1.0"
}

0 comments on commit d42fcf5

Please sign in to comment.