Skip to content
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

fix: update bucket policy to work with permission boundaries #637

Merged
merged 3 commits into from
Nov 3, 2022
Merged

fix: update bucket policy to work with permission boundaries #637

merged 3 commits into from
Nov 3, 2022

Conversation

vmarinescu
Copy link
Contributor

Solves #635

The aim of this PR is to update the bucket policy so that it does not prevent downloading of the object in some cases.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@vmarinescu vmarinescu changed the title Improves bucket policy fix: Improves bucket policy Oct 25, 2022
@dontirun dontirun changed the title fix: Improves bucket policy fix: update bucket policy to work with permission boundaries Oct 31, 2022
@dontirun
Copy link
Contributor

fixes #532

@vmarinescu
Copy link
Contributor Author

One check is failing, but not sure why. Should I merge main branch into my branch and push again?

@dontirun dontirun self-requested a review November 3, 2022 15:28
@dontirun
Copy link
Contributor

dontirun commented Nov 3, 2022

@vmarinescu let's try that. Otherwise I think you may need to grant permissions on the branch for workflows and maintainers to push to it

@mergify mergify bot merged commit a0016ef into awslabs:main Nov 3, 2022
jpduckwo pushed a commit to jpduckwo/cdk-serverless-clamscan that referenced this pull request Sep 4, 2023
…#637)

Solves awslabs#635 

The aim of this PR is to update the bucket policy so that it does not prevent downloading of the object in some cases.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants