Fix: functions name when basePath is setup#345
Fix: functions name when basePath is setup#345framled wants to merge 33 commits intomilliHQ:v0.xfrom
Conversation
|
Hey, thanks for submitting this PR. If it's too confusing, please allow me to make the necessary changes, by Enabling repository maintainer permissions on existing pull requests. |
|
Hi @framled I think this is not about only the fixing the Lambda function name, it also requires the change in proxy. Have you got success to test end to end flow only by changing Lambda function name? |
|
@sumantapakira Yes it works |
|
ok, but with your change when I tried to create new infra then it fails - Error: error putting S3 Bucket Notification Configuration: InvalidArgument: Unable to validate the following destination configurations with module.tf_next.module.statics_deploy.aws_s3_bucket_notification.on_create, |
Terraform failed when basePath is set, this is because lambda functions name can not have a special character.
issues related to this are #288 #310