Skip to content
This repository has been archived by the owner on Aug 4, 2023. It is now read-only.

Commit

Permalink
fix: added tfsec rule to ignore public ingress warning (#620)
Browse files Browse the repository at this point in the history
* fix: added tfsec ignore rule to ignore public ingress warning

* fix: formatting issues

Co-authored-by: nabil.aziz <[email protected]>
  • Loading branch information
azizn03 and nabil.aziz authored Aug 19, 2022
1 parent a675ae2 commit 82c3c3e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions modules/environment/aws/eks/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -111,8 +111,7 @@ resource "aws_security_group" "worker" {
security_groups = [aws_security_group.elb.id]
}
}

#tfsec:ignore:aws-vpc-no-public-ingress-sg
#tfsec:ignore:aws-vpc-no-public-ingress-sgr
resource "aws_security_group" "elb" {
name_prefix = "${var.cluster}-ingress-elb"
vpc_id = data.aws_vpc.lead_vpc.id
Expand All @@ -134,6 +133,7 @@ resource "aws_security_group" "elb" {
"0.0.0.0/0",
]
}

ingress {
description = "Allow HTTPS access"
from_port = 443
Expand Down

0 comments on commit 82c3c3e

Please sign in to comment.