Merge pull request #2849 from kakakakakku/apigw-bedrock-nova-canvas

apigw-bedrock-nova-canvas: Small improvements (Fix Terraform warning and README)

Author: julianwood

apigw-bedrock-nova-canvas/README.md

@@ -22,8 +22,6 @@ Important: this application uses various AWS services and there are costs associ
 
 For this pattern, you would need access only to Amazon Nova Canvas foundation model (Model ID: amazon.nova-canvas-v1:0) in us-east-1 region, since the pattern uses us-east-1 region by default.
 
-You must request access to the model before you can use it. If you try to use the model before you have requested access to it, you will receive an error message.
-
 1. Create a new directory, navigate to that directory in a terminal and clone the GitHub repository:
     ``` 
     git clone https://github.com/aws-samples/serverless-patterns
@@ -86,4 +84,4 @@ You must request access to the model before you can use it. If you try to use th
 ----
 Copyright 2025 Amazon.com, Inc. or its affiliates. All Rights Reserved.
 
-SPDX-License-Identifier: MIT-0
+SPDX-License-Identifier: MIT-0

apigw-bedrock-nova-canvas/main.tf

@@ -36,7 +36,7 @@ resource "aws_iam_policy" "invoke_model_policy" {
         ]
         Effect = "Allow"
         Resource = [
-          "arn:aws:bedrock:${data.aws_region.current.name}::foundation-model/amazon.nova-canvas-v1:0"
+          "arn:aws:bedrock:${data.aws_region.current.region}::foundation-model/amazon.nova-canvas-v1:0"
         ]
       },
     ]
@@ -45,23 +45,23 @@ resource "aws_iam_policy" "invoke_model_policy" {
 
 # S3 bucket for storing images
 resource "aws_s3_bucket" "image_bucket" {
-  bucket = "${lower(var.prefix)}-image-bucket-${random_string.suffix.result}"
+  bucket        = "${lower(var.prefix)}-image-bucket-${random_string.suffix.result}"
   force_destroy = true
 }
 
 # Create CloudWatch Log Group for Lambda
 resource "aws_cloudwatch_log_group" "lambda_log_group" {
   name              = "/aws/lambda/${lower(var.prefix)}-invoke-bedrock"
   retention_in_days = 14
-  
+
   lifecycle {
     prevent_destroy = false
   }
 }
 
 # Lambda function
 resource "aws_lambda_function" "invoke_bedrock_function" {
-  filename      = "index.zip"  # Replace with your Lambda code zip file
+  filename      = "index.zip" # Replace with your Lambda code zip file
   function_name = "${lower(var.prefix)}-invoke-bedrock"
   role          = aws_iam_role.lambda_role.arn
   handler       = "index.handler"
@@ -146,7 +146,7 @@ resource "aws_api_gateway_integration" "lambda_integration" {
 # API Gateway Deployment
 resource "aws_api_gateway_deployment" "api_deployment" {
   rest_api_id = aws_api_gateway_rest_api.bedrock_api.id
-  
+
   depends_on = [
     aws_api_gateway_integration.lambda_integration
   ]
@@ -176,12 +176,12 @@ output "lambda_function" {
 
 output "api_endpoint" {
   description = "The API Gateway endpoint URL "
-  value = "${aws_api_gateway_stage.api_stage.invoke_url}/image_gen"
+  value       = "${aws_api_gateway_stage.api_stage.invoke_url}/image_gen"
 }
 
 output "s3_image_bucket" {
   description = "The Output S3 bucket is "
-  value = aws_s3_bucket.image_bucket.id
+  value       = aws_s3_bucket.image_bucket.id
 }
 
 # Data source for current region