Update template.yaml to use BucketPolicy

[sambler]

Replace legacy S3 AccessControl with a bucket policy.

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

[sambler]

ignore the noise, there is already a bucket policy in place that works, so bring this down to removing the legacy AccessControl

[ottokruse]

Sure, thanks. What's the benefit of not specifying private? Just cleaner (fine reason) as ACLs are deprecated, or will there be an actual effect too?

[sambler]

At the time I got a fail to create bucket, needed to remove it to create the stack.

A quick test of only a bucket works with sam and cfn, so it may have been a bucket policy I added that conflicted with the Accesscontrol.

[ottokruse]

Okay sure. And what's the benefit of removing AccessControl=Private now from the template? What will that improve?

I think it's a good idea because that property is deprecated but just wondering if it has a tangible benefit besides that.

[sambler]

Turns out it would only make it easier to add other bucket policies, so not directly effecting this stack now.

I only submitted because I thought having the AccessControl was preventing the stack create, which I found to be a conflict with my modifications.

[ottokruse]

Aight, well let's get rid of a deprecated property 🤘🏻