Skip to content

Commit

Permalink
bug: aws security hub in management settings need to be remove to pre…
Browse files Browse the repository at this point in the history
…vent cyclic dependency (#186)
  • Loading branch information
marwinbaumannsbp authored Jul 25, 2023
1 parent c4ac0b5 commit 16ec417
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions security_hub.tf
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,7 @@ resource "aws_securityhub_organization_admin_account" "default" {
}

resource "aws_securityhub_account" "management" {
auto_enable_controls = var.aws_security_hub.auto_enable_controls
control_finding_generator = var.aws_security_hub.control_finding_generator
enable_default_standards = var.aws_security_hub.auto_enable_default_standards

depends_on = [aws_securityhub_organization_configuration.default]
}
Expand Down Expand Up @@ -37,9 +35,7 @@ resource "aws_securityhub_standards_subscription" "management" {
resource "aws_securityhub_account" "default" {
provider = aws.audit

auto_enable_controls = var.aws_security_hub.auto_enable_controls
control_finding_generator = var.aws_security_hub.control_finding_generator
enable_default_standards = var.aws_security_hub.auto_enable_default_standards
}

resource "aws_securityhub_organization_configuration" "default" {
Expand Down

0 comments on commit 16ec417

Please sign in to comment.