Skip to content

Create bucket.tf - #4

Open
bahar-shah wants to merge 1 commit into
masterfrom
bahar-shah-patch-3
Open

Create bucket.tf#4
bahar-shah wants to merge 1 commit into
masterfrom
bahar-shah-patch-3

Conversation

@bahar-shah

Copy link
Copy Markdown
Owner

Closes #

Reason for Proposed Changes

Proposed Changes

I submit this contribution under the Apache-2.0 license.

@github-actions

Copy link
Copy Markdown

kics-logo

KICS version: v2.1.7

Category Results
CRITICAL CRITICAL 0
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 1
Files parsed placeholder 1
Files failed to scan placeholder 0
Total executed queries placeholder 47
Queries failed to execute placeholder 0
Execution time placeholder 0

Comment thread demo/bucket.tf
Comment on lines +14 to +15
resource "aws_s3_bucket" "positive1" {
bucket = "my-test-bucket"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 Infrastructure as Code Configuration Issue

'logging' is undefined or null (...read more)

[f861041c-8c9f-4156-acfc-5e6e524f5884] Server Access Logging should be enabled on S3 Buckets so that all changes are logged and trackable

View in Datadog  Leave us feedback  Documentation

Comment thread demo/bucket.tf
Comment on lines +25 to +26
enabled = false
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 Infrastructure as Code Configuration Issue

'versioning.enabled' is set to false (...read more)

[568a4d22-3517-44a6-a7ad-6a7eed88722c] S3 bucket should have versioning enabled

View in Datadog  Leave us feedback  Documentation

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