Skip to content
This repository was archived by the owner on Jan 10, 2023. It is now read-only.

Remove public-read header#302

Merged
drazisil merged 2 commits intomasterfrom
remove-public-read-header
May 18, 2020
Merged

Remove public-read header#302
drazisil merged 2 commits intomasterfrom
remove-public-read-header

Conversation

@hootener
Copy link

@hootener hootener commented May 18, 2020

Purpose

Changes to codecov uploading infrastructure mean the public-read header is no longer required. This PR removes the header.

Notable Changes

Removal of X-Amz-Acl: public-read

Update the SHA1SUM file

636a422

eliatcodecov and others added 2 commits May 18, 2020 08:34
Changes to codecov uploading infrastructure mean the public-read header is no longer required.
@hootener hootener requested a review from drazisil May 18, 2020 13:39
@drazisil drazisil added bug needs deployment This PR will need a new deployment labels May 18, 2020
Copy link
Contributor

@drazisil drazisil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@drazisil drazisil merged commit e957ed9 into master May 18, 2020
@drazisil
Copy link
Contributor

Waiting on deployment

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug needs deployment This PR will need a new deployment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants