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

C++: Update hello_s3.cpp #5254

Merged
merged 1 commit into from
Aug 14, 2023
Merged

C++: Update hello_s3.cpp #5254

merged 1 commit into from
Aug 14, 2023

Conversation

annapan-aws
Copy link
Contributor

Adding explicit authentication check. S3 allows anonymous requests. Due to an issue with the c++ sdk, if a user is not authenticated this method will return an unhelpful ‘0 buckets’ instead of a helpful ‘authentication issue’ message. Since this example is for new users, lets be more helpful. See https://sim.amazon.com/issues/search?q=status%3A(Open)+containingFolder%3A(f50eb8c1-3469-4bb5-a16e-a6535f0b3c31)+(annapan)&sort=lastUpdatedConversationDate+desc&selectedDocument=afec5f96-00d4-4c4e-88c7-84179f16feaf&id_token=eyJ0eXAiOiJKV1MiLCJhbGciOiJQUzI1NiIsIng1dSI6Imh0dHA6Ly9zZW50cnktcGtpLmFtYXpvbi5jb20vcHVibGlja2V5LzEzNDM4NTk4In0.eyJpc3MiOiJzZW50cnkuYW1hem9uLmNvbSIsInN1YiI6ImFubmFwYW5AQU5ULkFNQVpPTi5DT00iLCJhdWQiOlsiaHR0cHM6Ly9zaW0uYW1hem9uLmNvbTo0NDMiXSwiZXhwIjoxNjkxNTk2NTYyLCJpYXQiOjE2OTE1OTU2NjIsImF1dGhfdGltZSI6MTY5MTU4ODQwNSwibm9uY2UiOiI3N2M2MWI1NzU0M2VkYWQ5MDQ1ZTZkMWZjZGI1MmIwYjRkOTllYzQ4NzA2NjJjODkwZTU4Y2QzMmIzOTFhZDM3IiwiYW1yIjpbIm1mYSIsIm90cCIsInB3ZCJdLCJyZXFfaWQiOm51bGwsImp0aSI6Img4NFU0YWJ4R05IUmU4bVpiUzExVUE9PSIsInBvc3R1cmVfY2hlY2siOjB9.DCFinQSQh7xSUEndVqDtmvxsx2waD3_tjIuIooaMSO_15oZk7UNoFG-2ZJ-kaqNTKT2dp7tM3QN65QN0qm21-vDtOxRtzCcuxAEOU4mXpsUaUFKRMOIbXuKQsIbj_t8O65IP16x64jMYjzDyrB7Ojm75pcRFZGq8rMypbLT-KwTGMFDMv8e1aXDYW1G0i84-4e9ITy9vTYnABrhG8AsgmEQe1g5UANg9G2kpkxzhGOqA4fedxKiAoMaXypxrl1rvBUKMlBtj_oxqQy1m97twrw-zVNfaYBmRqyv2BropFkJQn5PjH4GyW9kWpbyMTEHwc5SCGzpa1HqI0_SwFC-lAg and https://quip-amazon.com/u7fzAhoT24rd/Anonymous-Auth-in-the-SDKs-with-S3 for more information.

This pull request...


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

@github-actions github-actions bot added the C++ This issue relates to the AWS SDK for C++ label Aug 11, 2023
@meyertst-aws meyertst-aws self-assigned this Aug 14, 2023
@meyertst-aws meyertst-aws added the Text Review not needed This contains no text which needs review label Aug 14, 2023
@meyertst-aws meyertst-aws added the On Call Review needed This work needs an on-call review label Aug 14, 2023
@rlhagerm rlhagerm changed the title Update hello_s3.cpp C++: Update hello_s3.cpp Aug 14, 2023
@rlhagerm rlhagerm added On Call Review complete On call review complete and removed On Call Review needed This work needs an on-call review labels Aug 14, 2023
Adding explicit authentication check. S3 allows anonymous requests. Due to an issue with the c++ sdk, if a user is not authenticated this method will return an unhelpful ‘0 buckets’ instead of a helpful ‘authentication issue’ message.  Since this example is for new users, lets be more helpful.  See https://sim.amazon.com/issues/search?q=status%3A(Open)+containingFolder%3A(f50eb8c1-3469-4bb5-a16e-a6535f0b3c31)+(annapan)&sort=lastUpdatedConversationDate+desc&selectedDocument=afec5f96-00d4-4c4e-88c7-84179f16feaf&id_token=eyJ0eXAiOiJKV1MiLCJhbGciOiJQUzI1NiIsIng1dSI6Imh0dHA6Ly9zZW50cnktcGtpLmFtYXpvbi5jb20vcHVibGlja2V5LzEzNDM4NTk4In0.eyJpc3MiOiJzZW50cnkuYW1hem9uLmNvbSIsInN1YiI6ImFubmFwYW5AQU5ULkFNQVpPTi5DT00iLCJhdWQiOlsiaHR0cHM6Ly9zaW0uYW1hem9uLmNvbTo0NDMiXSwiZXhwIjoxNjkxNTk2NTYyLCJpYXQiOjE2OTE1OTU2NjIsImF1dGhfdGltZSI6MTY5MTU4ODQwNSwibm9uY2UiOiI3N2M2MWI1NzU0M2VkYWQ5MDQ1ZTZkMWZjZGI1MmIwYjRkOTllYzQ4NzA2NjJjODkwZTU4Y2QzMmIzOTFhZDM3IiwiYW1yIjpbIm1mYSIsIm90cCIsInB3ZCJdLCJyZXFfaWQiOm51bGwsImp0aSI6Img4NFU0YWJ4R05IUmU4bVpiUzExVUE9PSIsInBvc3R1cmVfY2hlY2siOjB9.DCFinQSQh7xSUEndVqDtmvxsx2waD3_tjIuIooaMSO_15oZk7UNoFG-2ZJ-kaqNTKT2dp7tM3QN65QN0qm21-vDtOxRtzCcuxAEOU4mXpsUaUFKRMOIbXuKQsIbj_t8O65IP16x64jMYjzDyrB7Ojm75pcRFZGq8rMypbLT-KwTGMFDMv8e1aXDYW1G0i84-4e9ITy9vTYnABrhG8AsgmEQe1g5UANg9G2kpkxzhGOqA4fedxKiAoMaXypxrl1rvBUKMlBtj_oxqQy1m97twrw-zVNfaYBmRqyv2BropFkJQn5PjH4GyW9kWpbyMTEHwc5SCGzpa1HqI0_SwFC-lAg and https://quip-amazon.com/u7fzAhoT24rd/Anonymous-Auth-in-the-SDKs-with-S3 for more information.
@rlhagerm rlhagerm merged commit 33e3086 into awsdocs:main Aug 14, 2023
8 checks passed
shepazon pushed a commit to shepazon/aws-doc-sdk-examples that referenced this pull request Aug 14, 2023
Update hello_s3.cpp

Adding explicit authentication check. S3 allows anonymous requests. Due to an issue with the c++ sdk, if a user is not authenticated this method will return an unhelpful ‘0 buckets’ instead of a helpful ‘authentication issue’ message.  Since this example is for new users, lets be more helpful.
meyertst-aws pushed a commit to meyertst-aws/aws-doc-sdk-examples that referenced this pull request Aug 15, 2023
Update hello_s3.cpp

Adding explicit authentication check. S3 allows anonymous requests. Due to an issue with the c++ sdk, if a user is not authenticated this method will return an unhelpful ‘0 buckets’ instead of a helpful ‘authentication issue’ message.  Since this example is for new users, lets be more helpful.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ This issue relates to the AWS SDK for C++ On Call Review complete On call review complete Text Review not needed This contains no text which needs review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants