Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Sets Bucket ACL after its ownership controls were set #177

Closed
wants to merge 1 commit into from
Closed

Sets Bucket ACL after its ownership controls were set #177

wants to merge 1 commit into from

Conversation

danfsd
Copy link

@danfsd danfsd commented Apr 27, 2023 •
edited
Loading

what

  • makes ACL to be defined after the Ownership Control is set

why

  • AWS changed the default value for Ownership Controls to BucketOwnerEnforced and this conflicts with having an ACL set on the bucket

references

@danfsd danfsd requested review from a team as code owners April 27, 2023 18:05
@danfsd danfsd requested review from srhopkins and florian0410 April 27, 2023 18:05
@danfsd danfsd mentioned this pull request Apr 27, 2023
Closed
@danfsd
Copy link
Author

danfsd commented Apr 27, 2023

ugh, did not see #176 before opening this PR, I think we should close this one in favor to it

@danfsd danfsd mentioned this pull request May 1, 2023
Closed
@danfsd
Copy link
Author

danfsd commented May 1, 2023

Closing in favor of either #175 or #176

@danfsd danfsd closed this May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@srhopkins srhopkins Awaiting requested review from srhopkins srhopkins was automatically assigned from cloudposse/contributors

@florian0410 florian0410 Awaiting requested review from florian0410 florian0410 was automatically assigned from cloudposse/contributors

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@danfsd