Skip to content

Add AWS anonymous access when reading/writing binary cache to AWS S3.#434

Merged
BillyONeal merged 1 commit intomicrosoft:mainfrom
AenBleidd:vko_add_aws_s3_anonymous_support
Mar 18, 2022
Merged

Add AWS anonymous access when reading/writing binary cache to AWS S3.#434
BillyONeal merged 1 commit intomicrosoft:mainfrom
AenBleidd:vko_add_aws_s3_anonymous_support

Conversation

@AenBleidd
Copy link
Contributor

Sometimes it's essential to be able to read/write binary cache to AWS S3 anonymously (mostly read, I assume).
AWS cli tool requires to be run with --no-sign-request option to get anonymous access.

This PR adds new 'x-aws-config' option to configure AWS anonymous access.

Signed-off-by: Vitalii Koshura lestat.de.lionkur@gmail.com

@AenBleidd AenBleidd force-pushed the vko_add_aws_s3_anonymous_support branch from f92d3f5 to 33f8b29 Compare March 14, 2022 00:28
@BillyONeal
Copy link
Member

BillyONeal commented Mar 16, 2022

I'm a bit nervous to make a change like this without an understanding of AWS's security model. Can you describe the potential threats to which a user is exposed if they pass this option?

@AenBleidd
Copy link
Contributor Author

User should enable anonymous access to the bucket manually and set exactly what can be done. For example, I set that anyone can list files in the bucket and download any file. But only authorized users (that are chosen manually) can write objects to the file.

In common case, if user set this option in vcpkg without properly configure AWS bucket to enable anonymous access - nothing will be enabled by default.

So this is completely safe option by default.

@AenBleidd AenBleidd force-pushed the vko_add_aws_s3_anonymous_support branch 2 times, most recently from dce1c92 to 09d2446 Compare March 16, 2022 22:57
Sometimes it's essential to be able to read/write binary cache to AWS S3 anonymously (mostly read, I assume).
AWS cli tool requires to be run with --no-sign-request option to get anonymous access.

This PR adds new 'x-aws-config' option to configure AWS anonymous access.

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
@AenBleidd AenBleidd force-pushed the vko_add_aws_s3_anonymous_support branch from 09d2446 to b7a7a1c Compare March 16, 2022 23:21
@AenBleidd AenBleidd requested a review from vicroms March 18, 2022 08:52
@BillyONeal
Copy link
Member

Twitter Driven Development also indicates that this isn't scary: https://twitter.com/sarbjeetjohal/status/1504727772958113797

Thanks for your contribution!

@BillyONeal BillyONeal merged commit abbdbbf into microsoft:main Mar 18, 2022
@AenBleidd
Copy link
Contributor Author

@BillyONeal, thank you!

@AenBleidd AenBleidd deleted the vko_add_aws_s3_anonymous_support branch March 18, 2022 22:50
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.

3 participants