Skip to content
This repository was archived by the owner on May 23, 2024. It is now read-only.

Commit 45ed571

Browse files
authored
Remove boto 'set_acl' call after upload (#49)
Since it causes access denied errors when the IAM user does not have `PutObjectAcl` permissions for the S3 bucket, preventing the use of a more locked down IAM user for deployment. It seems preferable to leave the bucket ACLs to the bucket owner, and for `bob deploy` to only upload the file. Closes W-8134681.
1 parent 7c172f3 commit 45ed571

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

bob/models.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,6 +192,5 @@ def deploy(self, allow_overwrite=False):
192192

193193
# Upload the archive, set permissions.
194194
key.set_contents_from_filename(self.archived_path)
195-
key.set_acl('public-read')
196195

197196
print_stderr('Upload complete!')

0 commit comments

Comments
 (0)