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.

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

s3_bucket - Ensure public_access settings are configured before policies #1511

Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented May 4, 2023

SUMMARY

At the end of April Amazon updated various S3 bucket defaults. Buckets now have public_access blocked by default, and object_owner set to "BucketOwnerEnforced".
https://aws.amazon.com/blogs/aws/heads-up-amazon-s3-security-changes-are-coming-in-april-of-2023/

This uncovered a race condition where we set the policy before setting the public_access configs.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

s3_bucket

ADDITIONAL INFORMATION

@tremble tremble added this to the 5.5.0 milestone May 4, 2023
@tremble tremble marked this pull request as ready for review May 4, 2023 09:37
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/eab647adb2c248cd92dcc2a1ef14bc3e

✔️ ansible-galaxy-importer SUCCESS in 3m 44s
✔️ build-ansible-collection SUCCESS in 17m 29s
✔️ ansible-test-splitter SUCCESS in 5m 19s
✔️ integration-amazon.aws-1 SUCCESS in 9m 05s
✔️ integration-community.aws-1 SUCCESS in 13m 29s
✔️ integration-community.aws-2 SUCCESS in 9m 07s
✔️ integration-community.aws-3 SUCCESS in 10m 08s
Skipped 40 jobs

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label May 4, 2023
@github-actions
Copy link

github-actions bot commented May 4, 2023

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/8c16d5f332fc42e9bda84bf7b9de47fe

✔️ ansible-galaxy-importer SUCCESS in 4m 44s
✔️ build-ansible-collection SUCCESS in 18m 13s
✔️ ansible-test-splitter SUCCESS in 8m 48s
✔️ integration-amazon.aws-1 SUCCESS in 7m 21s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit f14f3ee into ansible-collections:main May 4, 2023
tremble added a commit to tremble/amazon.aws that referenced this pull request May 4, 2023
…ies (ansible-collections#1511)

s3_bucket - Ensure public_access settings are configured before policies

SUMMARY
At the end of April Amazon updated various S3 bucket defaults.  Buckets now have public_access blocked by default, and object_owner set to "BucketOwnerEnforced".
https://aws.amazon.com/blogs/aws/heads-up-amazon-s3-security-changes-are-coming-in-april-of-2023/
This uncovered a race condition where we set the policy before setting the public_access configs.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_bucket
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
softwarefactory-project-zuul bot pushed a commit that referenced this pull request May 4, 2023
…ies (#1511) (#1512)

[stable-5] [backport/1511] s3_bucket - Ensure public_access settings are configured before policies

Manual backport of #1511
SUMMARY
At the end of April Amazon updated various S3 bucket defaults.  Buckets now have public_access blocked by default, and object_owner set to "BucketOwnerEnforced". https://aws.amazon.com/blogs/aws/heads-up-amazon-s3-security-changes-are-coming-in-april-of-2023/ This uncovered a race condition where we set the policy before setting the public_access configs
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_bucket
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
softwarefactory-project-zuul bot pushed a commit to ansible-collections/community.aws that referenced this pull request May 4, 2023
Fix s3_sync and s3_logging integration tests

SUMMARY
See also: ansible-collections/amazon.aws#1511
At the end of April Amazon updated various S3 bucket defaults. Buckets now have public_access blocked by default, and object_owner set to "BucketOwnerEnforced".
https://aws.amazon.com/blogs/aws/heads-up-amazon-s3-security-changes-are-coming-in-april-of-2023/
This change to the defaults resulted in some of our tests failing.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_sync
s3_logging
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
patchback bot pushed a commit to ansible-collections/community.aws that referenced this pull request May 4, 2023
Fix s3_sync and s3_logging integration tests

SUMMARY
See also: ansible-collections/amazon.aws#1511
At the end of April Amazon updated various S3 bucket defaults. Buckets now have public_access blocked by default, and object_owner set to "BucketOwnerEnforced".
https://aws.amazon.com/blogs/aws/heads-up-amazon-s3-security-changes-are-coming-in-april-of-2023/
This change to the defaults resulted in some of our tests failing.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_sync
s3_logging
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
(cherry picked from commit 4cad9ee)
tremble added a commit to ansible-collections/community.aws that referenced this pull request May 4, 2023
Fix s3_sync and s3_logging integration tests

SUMMARY
See also: ansible-collections/amazon.aws#1511
At the end of April Amazon updated various S3 bucket defaults. Buckets now have public_access blocked by default, and object_owner set to "BucketOwnerEnforced".
https://aws.amazon.com/blogs/aws/heads-up-amazon-s3-security-changes-are-coming-in-april-of-2023/
This change to the defaults resulted in some of our tests failing.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_sync
s3_logging
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
(cherry picked from commit 4cad9ee)

Co-authored-by: Mark Chappell <[email protected]>
@tremble tremble deleted the s3/acls branch September 27, 2023 07:00
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants