-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Corrected AWS KMS Key policy to attach the AWS CloudWatch log group #33
Conversation
kms.tf
Outdated
resource "aws_kms_key" "encryption_rest" { | ||
enable_key_rotation = true | ||
description = "Key to encrypt cache at rest." | ||
deletion_window_in_days = 7 | ||
#checkov:skip=CKV2_AWS_64: Not including a KMS Key policy | ||
} |
Check warning
Code scanning / checkov
Ensure KMS key Policy is defined Warning
resource "aws_kms_key" "encryption_secret" { | ||
enable_key_rotation = true | ||
description = "Key to encrypt secret" | ||
deletion_window_in_days = 7 | ||
#checkov:skip=CKV2_AWS_64: Not including a KMS Key policy | ||
} |
Check warning
Code scanning / checkov
Ensure KMS key Policy is defined Warning
Infracost report💰 Monthly cost will increase by $223 📈
Cost details
|
Terraform Format and Style 🖌
|
Terraform Format and Style 🖌
|
Terraform Format and Style 🖌
|
The changes in this PR closes #32 and #30
These are the changes:
-correct the KMS key attached to the AWS CloudWatch log group logs
-attach a KMS key policy to the KMS key
-correct the KMS key names