Skip to content

Commit 4fee301

Browse files
authored
feat: No defaults rules (#3)
* remove default egress rule * add sample ingress rule * docs
1 parent c364b6b commit 4fee301

File tree

2 files changed

+15
-6
lines changed

2 files changed

+15
-6
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ No modules.
5454
|------|-------------|------|---------|:--------:|
5555
| <a name="input_common_tags"></a> [common\_tags](#input\_common\_tags) | A map of tags to assign to every resource in this module. | `map(string)` | `{}` | no |
5656
| <a name="input_description"></a> [description](#input\_description) | Security group description. | `string` | `null` | no |
57-
| <a name="input_egress_rules"></a> [egress\_rules](#input\_egress\_rules) | Outbound rules (egress) for this security group. | <pre>map(object({<br> description = optional(string, null)<br> ip_protocol = optional(string, "tcp")<br><br> port_number = optional(number, null)<br> from_port = optional(number, null)<br> to_port = optional(number, null)<br><br> icmp_type = optional(number, null)<br> icmp_code = optional(number, null)<br><br> cidr_ipv4 = optional(string, null)<br> cidr_ipv6 = optional(string, null)<br> prefix_list_id = optional(string, null)<br> security_group_id = optional(string, null)<br> extra_tags = optional(map(string), {})<br> }))</pre> | <pre>{<br> "allow-all": {<br> "cidr_ipv4": "0.0.0.0/0",<br> "ip_protocol": "all"<br> }<br>}</pre> | no |
57+
| <a name="input_egress_rules"></a> [egress\_rules](#input\_egress\_rules) | Outbound rules (egress) for this security group. | <pre>map(object({<br> description = optional(string, null)<br> ip_protocol = optional(string, "tcp")<br><br> port_number = optional(number, null)<br> from_port = optional(number, null)<br> to_port = optional(number, null)<br><br> icmp_type = optional(number, null)<br> icmp_code = optional(number, null)<br><br> cidr_ipv4 = optional(string, null)<br> cidr_ipv6 = optional(string, null)<br> prefix_list_id = optional(string, null)<br> security_group_id = optional(string, null)<br> extra_tags = optional(map(string), {})<br> }))</pre> | `{}` | no |
5858
| <a name="input_ingress_rules"></a> [ingress\_rules](#input\_ingress\_rules) | Inbound rules (ingress) for this security group. | <pre>map(object({<br> description = optional(string, null)<br> ip_protocol = optional(string, "tcp")<br><br> port_number = optional(number, null)<br> from_port = optional(number, null)<br> to_port = optional(number, null)<br><br> icmp_type = optional(number, null)<br> icmp_code = optional(number, null)<br><br> cidr_ipv4 = optional(string, null)<br> cidr_ipv6 = optional(string, null)<br> prefix_list_id = optional(string, null)<br> security_group_id = optional(string, null)<br> extra_tags = optional(map(string), {})<br> }))</pre> | `{}` | no |
5959
| <a name="input_name"></a> [name](#input\_name) | Name of the security group. | `string` | `null` | no |
6060
| <a name="input_vpc_id"></a> [vpc\_id](#input\_vpc\_id) | The ID of the VPC. | `string` | n/a | yes |

variables.tf

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,16 @@ variable "ingress_rules" {
4242
security_group_id = optional(string, null)
4343
extra_tags = optional(map(string), {})
4444
}))
45-
default = {}
45+
default = {
46+
# "icmp-in" = {
47+
# ip_protocol = "icmp"
48+
# cidr_ipv4 = "0.0.0.0/0"
49+
# }
50+
# "ssh-in" = {
51+
# cidr_ipv4 = "0.0.0.0/0"
52+
# port_number = 22
53+
# }
54+
}
4655
}
4756

4857
variable "egress_rules" {
@@ -66,9 +75,9 @@ variable "egress_rules" {
6675
extra_tags = optional(map(string), {})
6776
}))
6877
default = {
69-
"allow-all" = {
70-
ip_protocol = "all"
71-
cidr_ipv4 = "0.0.0.0/0"
72-
}
78+
# "allow-all" = {
79+
# ip_protocol = "all"
80+
# cidr_ipv4 = "0.0.0.0/0"
81+
# }
7382
}
7483
}

0 commit comments

Comments
 (0)