-
Notifications
You must be signed in to change notification settings - Fork 152
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: functions name when basePath is setup #345
base: v0.x
Are you sure you want to change the base?
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