cloudposse / terraform-aws-transfer-sftp

https://cloudposse.com/accelerate
Apache License 2.0
29 stars 48 forks source link

Support multiple S3 buckets #25

Closed nitrocode closed 2 years ago

nitrocode commented 2 years ago

Have a question? Please checkout our Slack Community or visit our Slack Archive.

Slack Community

Describe the Feature

It would be good to support multiple S3 buckets. At the moment, this will take a single s3_bucket_name and add that to the IAM policy which is attached to the IAM role created by this module.

A workaround could be to output the IAM role arn/name in this module and then we can attach policies from outside this module to the IAM role that it created.

Looks like the following output could be used

https://github.com/cloudposse/terraform-aws-transfer-sftp/blob/6f3e1b98345f93a49cf0fe288d4b3eceb81dd4e4/outputs.tf#L16-L19