Skip to content

Conversation

@kakakakakku
Copy link
Contributor

Issue #, if available:
N/A

Description of changes:
Hi😀 Thanks for the useful patterns!

This pattern uses the NODEJS_14_X runtime on AWS Lambda and can no longer be deployed. Additionally, AWS CDK v1 has been retired.

So I updated to the NODEJS_22_X runtime and AWS CDK v2.

Check

cdk deploy completed successfully and works good.

$ curl -s https://8fgy48gsjb.execute-api.ap-northeast-1.amazonaws.com/prod
Success path: "/"

$ curl -s https://8fgy48gsjb.execute-api.ap-northeast-1.amazonaws.com/prod/hello
Success path: "/hello"

Thank you😀


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

new CfnWebACLAssociation(this, "APIGatewayWebACLAssociation", {
webAclArn: APIGatewayWebACL.attrArn,
resourceArn: cdk.Fn.join("", ["arn:aws:apigateway:us-east-1::/restapis/", cdk.Fn.importValue("demorestapiid"), "/stages/prod", ])
resourceArn: cdk.Fn.join("", ["arn:aws:apigateway:", this.region, "::/restapis/", cdk.Fn.importValue("demorestapiid"), "/stages/prod"])
Copy link
Contributor Author

Choose a reason for hiding this comment

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

note: It was previously fixed to the us-east-1 region, but I’ve updated it to support other regions. I tested it in ap-northeast-1.

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.

3 participants