refactor: fix failing warnings s3 bucket configuration (#2)

Author: damianKokot

.version

@@ -1 +1 @@
-v1.0.0
+v1.0.1

README.md

@@ -21,7 +21,10 @@
 | [aws_kms_alias.state](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/kms_alias) | resource |
 | [aws_kms_key.state](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/kms_key) | resource |
 | [aws_s3_bucket.bucket](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket) | resource |
+| [aws_s3_bucket_acl.acl](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket_acl) | resource |
 | [aws_s3_bucket_public_access_block.bucket-acl](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket_public_access_block) | resource |
+| [aws_s3_bucket_server_side_encryption_configuration.state_encryption](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket_server_side_encryption_configuration) | resource |
+| [aws_s3_bucket_versioning.state_versioning](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/s3_bucket_versioning) | resource |
 ## Inputs
 
 | Name | Description | Type | Default | Required |
@@ -43,7 +46,7 @@
 ### Basic example
 ```hcl
 module "string_ssm_keys" {
-  source              = "[email protected]:masterborn/terraform-kms-tfstate-backend.git?ref=v1.0.0"
+  source              = "[email protected]:masterborn/terraform-kms-tfstate-backend.git?ref=v1.0.1"
   bucket_name         = "example-state-bucket"
   dynamodb_table_name = "example-state-lock"
 

main.tf

@@ -24,27 +24,35 @@ resource "aws_kms_alias" "state" {
 
 resource "aws_s3_bucket" "bucket" {
   bucket = var.bucket_name
+  tags   = local.tags
+
+  lifecycle {
+    prevent_destroy = true
+  }
+}
+
+resource "aws_s3_bucket_acl" "acl" {
+  bucket = aws_s3_bucket.bucket.id
   acl    = "private"
+}
 
+resource "aws_s3_bucket_versioning" "state_versioning" {
+  bucket = aws_s3_bucket.bucket.id
 
-  versioning {
-    enabled = true
+  versioning_configuration {
+    status = "Enabled"
   }
+}
 
-  lifecycle {
-    prevent_destroy = true
-  }
+resource "aws_s3_bucket_server_side_encryption_configuration" "state_encryption" {
+  bucket = aws_s3_bucket.bucket.id
 
-  server_side_encryption_configuration {
-    rule {
-      apply_server_side_encryption_by_default {
-        kms_master_key_id = aws_kms_key.state.arn
-        sse_algorithm     = "aws:kms"
-      }
+  rule {
+    apply_server_side_encryption_by_default {
+      kms_master_key_id = aws_kms_key.state.arn
+      sse_algorithm     = "aws:kms"
     }
   }
-
-  tags = local.tags
 }
 
 resource "aws_s3_bucket_public_access_block" "bucket-acl" {