Skip to content

Commit 2d4a2ca

Browse files
authored
Merge pull request #23543 from crazy-max/ci-s3-no-public-acl
ci: don't set public acl when uploading files to s3 bucket
2 parents 4a0ea3c + e3677a4 commit 2d4a2ca

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

.github/workflows/deploy.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,6 @@ jobs:
100100
if: ${{ env.DOCS_S3_BUCKET != '' }}
101101
run: |
102102
aws --region ${{ env.DOCS_AWS_REGION }} s3 sync \
103-
--acl public-read \
104103
--delete \
105104
--exclude "*" \
106105
--include "*.webp" \
@@ -109,7 +108,6 @@ jobs:
109108
--content-type="image/webp" \
110109
public s3://${{ env.DOCS_S3_BUCKET }}/
111110
aws --region ${{ env.DOCS_AWS_REGION }} s3 sync \
112-
--acl public-read \
113111
--delete \
114112
--exclude "*.webp" \
115113
public s3://${{ env.DOCS_S3_BUCKET }}/

0 commit comments

Comments
 (0)