Skip to content

Commit e6573fc

Browse files
Reduce alert noise: Security Hub HIGH/CRITICAL only (#41)
## Summary Filter Security Hub EventBridge rule from MEDIUM+ to HIGH/CRITICAL only. A single S3 bucket create/delete was generating 6 Slack alerts: 1. Resource Created (useful) 2. Untagged Resource — Config rule (useful) 3. Resource Deleted (useful) 4. Security Hub: required-tags MEDIUM (duplicate of #2) 5. Security Hub: SSL not required MEDIUM (noise) 6. Security Hub: access logging not enabled MEDIUM (noise) After this change: alerts 4-6 are suppressed. Config + compliance reporter already cover tagging. ## Future work - Batch alerts into 5-minute windows - Deduplicate Config vs Security Hub findings - S3 account-wide defaults for access logging
1 parent 3248292 commit e6573fc

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

  • terraform/platform/monitoring

‎terraform/platform/monitoring/main.tf‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -440,18 +440,20 @@ resource "aws_securityhub_standards_subscription" "aws_foundational" {
440440
depends_on = [aws_securityhub_account.main]
441441
}
442442

443-
# Route Security Hub findings with severity >= MEDIUM to security SNS
443+
# Route Security Hub findings with severity >= HIGH to security SNS.
444+
# MEDIUM findings are too noisy (SSL, logging, tags) — these are covered
445+
# by Config rules and compliance reporter already.
444446
resource "aws_cloudwatch_event_rule" "securityhub_findings" {
445447
name = "${var.project}-securityhub-findings"
446-
description = "Route Security Hub findings (MEDIUM and above) to SNS"
448+
description = "Route Security Hub findings (HIGH and above) to SNS"
447449

448450
event_pattern = jsonencode({
449451
source = ["aws.securityhub"]
450452
detail-type = ["Security Hub Findings - Imported"]
451453
detail = {
452454
findings = {
453455
Severity = {
454-
Label = ["MEDIUM", "HIGH", "CRITICAL"]
456+
Label = ["HIGH", "CRITICAL"]
455457
}
456458
}
457459
}

0 commit comments

Comments
 (0)