fix(DMVP-4760): Add policy attachment support

main.tf

@@ -23,8 +23,8 @@ module "bucket" {
 
   website = var.website
 
-  policy        = local.is_public ? data.aws_iam_policy_document.public[0].json : ""
-  attach_policy = local.is_public
+  policy        = local.is_public ? data.aws_iam_policy_document.public[0].json : var.policy
+  attach_policy = local.is_public || var.attach_policy  // To Do: Add support for merging two policies
 }
 
 // have initial index.html file content

variables.tf

@@ -133,3 +133,15 @@ variable "event_notification_config" {
     events        = ["s3:ObjectCreated:*"]
   }
 }
+
+variable "policy" {
+  description = "Bucket policies to be attached to the bucket in case custom policies are needed for the bucket"
+  type        = any
+  default     = "" # Default case is empty, to be compatibile with the previous behavior
+}
+
+variable "attach_policy" {
+  description = "Flag to control if the policy should be attached to the bucket"
+  type        = bool
+  default     = false
+}