Skip to content
This repository has been archived by the owner on Oct 19, 2020. It is now read-only.

Commit

Permalink
MGMT-1785 Don't set extra ACL when uploading to S3 (#24)
Browse files Browse the repository at this point in the history
We don't have this permission in the integration environment
and I don't think we need to do this
  • Loading branch information
carbonin authored Aug 12, 2020
1 parent d51c640 commit 1fe7aa8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion render_files.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ def get_s3_client(s3_endpoint_url, aws_access_key_id, aws_secret_access_key):

def upload_to_aws(s3_client, local_file, bucket, s3_file):
try:
s3_client.upload_file(local_file, bucket, s3_file, ExtraArgs={'ACL': 'public-read'})
s3_client.upload_file(local_file, bucket, s3_file)
print("Upload Successful")
return True
except NoCredentialsError:
Expand Down

0 comments on commit 1fe7aa8

Please sign in to comment.