Skip to content

feat: add 4 Security Hub TLS checks (CKV_AWS_394-397) - #7661

Open
lourenco00 wants to merge 1 commit into
bridgecrewio:mainfrom
lourenco00:feat/security-hub-tls-checks
Open

feat: add 4 Security Hub TLS checks (CKV_AWS_394-397)#7661
lourenco00 wants to merge 1 commit into
bridgecrewio:mainfrom
lourenco00:feat/security-hub-tls-checks

Conversation

@lourenco00

Copy link
Copy Markdown

Summary

Adds 4 new Terraform resource checks for AWS Security Hub TLS controls that currently have no native Checkov coverage:

Security Hub Control Check ID Resource Logic
ELB.21 CKV_AWS_394 aws_lb_target_group, aws_alb_target_group Health check protocol must be HTTPS (skips lambda, TCP/UDP/GENEVE)
ELB.22 CKV_AWS_395 aws_lb_target_group, aws_alb_target_group Transport protocol must be HTTPS or TLS (skips lambda)
APIGateway.10 CKV_AWS_396 aws_apigatewayv2_integration VPC_LINK integrations must have tls_config with server_name_to_verify
DMS.12 CKV_AWS_397 aws_dms_endpoint Redis endpoints must not use plaintext (absent defaults to ssl-encryption)

Testing

  • Each check has a fixture directory with pass, fail, and not-applicable Terraform resources
  • Runner-based unit tests explicitly assert which resources pass and which fail
  • All 520 existing tests/terraform/checks/resource/aws/ tests continue to pass

AWS Config Rule Mappings

  • CKV_AWS_394 → elbv2-targetgroup-healthcheck-protocol-encrypted
  • CKV_AWS_395 → elbv2-targetgroup-protocol-encrypted
  • CKV_AWS_396 → Security Hub native control (no direct AWS Config equivalent)
  • CKV_AWS_397 → dms-redis-tls-enabled

Add new Terraform resource checks for AWS Security Hub TLS controls:

- CKV_AWS_394 (ELB.21): Target group health check protocol must be HTTPS
- CKV_AWS_395 (ELB.22): Target group transport protocol must be encrypted
- CKV_AWS_396 (APIGateway.10): API Gateway V2 VPC_LINK integrations must use tls_config
- CKV_AWS_397 (DMS.12): DMS Redis endpoints must have TLS enabled

Each check includes test fixtures and Runner-based unit tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant