-
Notifications
You must be signed in to change notification settings - Fork 4k
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
aws s3 sync --acl does not update the ACL of existing files on S3 #3215
Comments
Hi thanks for reporting. It seems like this issue is a dupe of the those related issues you linked. Was there something different in this issue you wanted to call out? I think the underlying problem is that the |
I think they are closely related but separate issues. They may well have the same root cause, but to the end-user the problem is slightly different. To highlight the differences between this issue and the others:
I think the bug report here stands well on its own since it is clear, focused, and comes with a minimal reproducer. |
Any update on this issue? |
1 similar comment
Any update on this issue? |
Marking as enhancement. It's slightly different than the linked issues but if we update to preserve ACL's, we will update to support updating ACL's based on the explicitly provided ACL. |
Bump |
It would be nice to have this upgrade. |
Bumpy bumpy |
Please try not to bump threads - just vote 👍 on the OP. Bumping threads sends notifications, it doesn't help teams prioritise. |
Just in case this is useful in the meantime...
|
* fix: add log streaming to `sam build` - IMAGE functions * fix: get closer to 95% test coverage * fix: address comments - Add type hinting - Add additional tests * fix: make mypy and lint happy * fix: address feedback
If you sync some files from local storage to S3 and then re-run the same sync command with a new
--acl
setting, you'd expect the ACLs of the existing objects on S3 to be updated. However, they are not.So, for example, after running these commands:
The objects in
s3://my-bucket/test/
will all have an ACL ofpublic-read
, when they should have an ACL ofprivate
.What I'm running:
Related: #901, #1060 (comment)
The text was updated successfully, but these errors were encountered: