Merge branch 'master' into master

Author: antonbabenko

CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file.
 
+## [5.5.0](https://github.com/terraform-aws-modules/terraform-aws-s3-bucket/compare/v5.4.0...v5.5.0) (2025-08-20)
+
+
+### Features
+
+* Add region parameter to notification module ([#354](https://github.com/terraform-aws-modules/terraform-aws-s3-bucket/issues/354)) ([2dd4364](https://github.com/terraform-aws-modules/terraform-aws-s3-bucket/commit/2dd4364b67d89cb9c881be465e5e4196ef8dea8f))
+
 ## [5.4.0](https://github.com/terraform-aws-modules/terraform-aws-s3-bucket/compare/v5.3.1...v5.4.0) (2025-08-11)
 
 

modules/notification/README.md

@@ -45,6 +45,7 @@ No modules.
 | <a name="input_create_sqs_policy"></a> [create\_sqs\_policy](#input\_create\_sqs\_policy) | Whether to create a policy for SQS permissions or not? | `bool` | `true` | no |
 | <a name="input_eventbridge"></a> [eventbridge](#input\_eventbridge) | Whether to enable Amazon EventBridge notifications | `bool` | `null` | no |
 | <a name="input_lambda_notifications"></a> [lambda\_notifications](#input\_lambda\_notifications) | Map of S3 bucket notifications to Lambda function | `any` | `{}` | no |
+| <a name="input_region"></a> [region](#input\_region) | Region where the resource(s) will be managed. Defaults to the region set in the provider configuration | `string` | `null` | no |
 | <a name="input_sns_notifications"></a> [sns\_notifications](#input\_sns\_notifications) | Map of S3 bucket notifications to SNS topic | `any` | `{}` | no |
 | <a name="input_sqs_notifications"></a> [sqs\_notifications](#input\_sqs\_notifications) | Map of S3 bucket notifications to SQS queue | `any` | `{}` | no |
 

modules/notification/main.tf

@@ -13,6 +13,8 @@ resource "aws_s3_bucket_notification" "this" {
 
   bucket = var.bucket
 
+  region = var.region
+
   eventbridge = var.eventbridge
 
   dynamic "lambda_function" {
@@ -62,6 +64,8 @@ resource "aws_s3_bucket_notification" "this" {
 resource "aws_lambda_permission" "allow" {
   for_each = { for k, v in var.lambda_notifications : k => v if var.create_lambda_permission }
 
+  region = var.region
+
   statement_id_prefix = "AllowLambdaS3BucketNotification-"
   action              = "lambda:InvokeFunction"
   function_name       = each.value.function_name
@@ -110,6 +114,8 @@ data "aws_iam_policy_document" "sqs" {
 resource "aws_sqs_queue_policy" "allow" {
   for_each = { for k, v in var.sqs_notifications : k => v if var.create_sqs_policy }
 
+  region = var.region
+
   queue_url = try(each.value.queue_id, local.queue_ids[each.key], null)
   policy    = data.aws_iam_policy_document.sqs[each.key].json
 }
@@ -145,6 +151,8 @@ data "aws_iam_policy_document" "sns" {
 resource "aws_sns_topic_policy" "allow" {
   for_each = { for k, v in var.sns_notifications : k => v if var.create_sns_policy }
 
+  region = var.region
+
   arn    = each.value.topic_arn
   policy = data.aws_iam_policy_document.sns[each.key].json
 }

modules/notification/variables.tf

@@ -22,6 +22,12 @@ variable "create_lambda_permission" {
   default     = true
 }
 
+variable "region" {
+  description = "Region where the resource(s) will be managed. Defaults to the region set in the provider configuration"
+  type        = string
+  default     = null
+}
+
 variable "bucket" {
   description = "Name of S3 bucket to use"
   type        = string

wrappers/notification/main.tf

@@ -11,6 +11,7 @@ module "wrapper" {
   create_sqs_policy        = try(each.value.create_sqs_policy, var.defaults.create_sqs_policy, true)
   eventbridge              = try(each.value.eventbridge, var.defaults.eventbridge, null)
   lambda_notifications     = try(each.value.lambda_notifications, var.defaults.lambda_notifications, {})
+  region                   = try(each.value.region, var.defaults.region, null)
   sns_notifications        = try(each.value.sns_notifications, var.defaults.sns_notifications, {})
   sqs_notifications        = try(each.value.sqs_notifications, var.defaults.sqs_notifications, {})
 }