Skip to content

Commit f7231da

Browse files
authored
Fix source_port_range when source_port_range is assigned * (#71)
* :0.13: * update * update * update * update
1 parent 2c6336f commit f7231da

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

main.tf

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ resource "azurerm_network_security_rule" "predefined_rules" {
2020
direction = element(var.rules[lookup(var.predefined_rules[count.index], "name")], 0)
2121
access = element(var.rules[lookup(var.predefined_rules[count.index], "name")], 1)
2222
protocol = element(var.rules[lookup(var.predefined_rules[count.index], "name")], 2)
23-
source_port_range = lookup(var.predefined_rules[count.index], "source_port_range", null ) == null ? "*" : null
24-
source_port_ranges = lookup(var.predefined_rules[count.index], "source_port_range", null ) != null ? split(",", var.predefined_rules[count.index].source_port_range) : null
23+
source_port_range = lookup(var.predefined_rules[count.index], "source_port_range", "*") == "*" ? "*" : null
24+
source_port_ranges = lookup(var.predefined_rules[count.index], "source_port_range", "*") == "*" ? null : split(",", var.predefined_rules[count.index].source_port_range)
2525
destination_port_range = element(var.rules[lookup(var.predefined_rules[count.index], "name")], 4)
2626
description = element(var.rules[lookup(var.predefined_rules[count.index], "name")], 5)
2727
source_address_prefix = lookup(var.predefined_rules[count.index], "source_application_security_group_ids", null) == null && var.source_address_prefixes == null ? join(",", var.source_address_prefix) : null
@@ -45,8 +45,8 @@ resource "azurerm_network_security_rule" "custom_rules" {
4545
direction = lookup(var.custom_rules[count.index], "direction", "Any")
4646
access = lookup(var.custom_rules[count.index], "access", "Allow")
4747
protocol = lookup(var.custom_rules[count.index], "protocol", "*")
48-
source_port_range = lookup(var.custom_rules[count.index], "source_port_range", null ) == null ? "*" : null
49-
source_port_ranges = lookup(var.custom_rules[count.index], "source_port_range", null ) != null ? split(",", var.custom_rules[count.index].source_port_range) : null
48+
source_port_range = lookup(var.custom_rules[count.index], "source_port_range", "*") == "*" ? "*" : null
49+
source_port_ranges = lookup(var.custom_rules[count.index], "source_port_range", "*") == "*" ? null : split(",", var.custom_rules[count.index].source_port_range)
5050
destination_port_ranges = split(",", replace(lookup(var.custom_rules[count.index], "destination_port_range", "*"), "*", "0-65535"))
5151
source_address_prefix = lookup(var.custom_rules[count.index], "source_application_security_group_ids", null) == null && lookup(var.custom_rules[count.index], "source_address_prefixes", null) == null ? lookup(var.custom_rules[count.index], "source_address_prefix", "*") : null
5252
source_address_prefixes = lookup(var.custom_rules[count.index], "source_application_security_group_ids", null) == null ? lookup(var.custom_rules[count.index], "source_address_prefixes", null) : null

0 commit comments

Comments
 (0)