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

docs(cognito_identity_credentials): Explain limitation of CognitoIden… #4455

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

RanVaknin
Copy link
Contributor

@RanVaknin RanVaknin commented Jun 22, 2023

This method uses the simplified authflow, which means users cannot explicitly set a session policy. This results in STS appending the Default Session Policy to the request, which can lead to limiting permissions even if they are explicitly attached to a role.

The doc change highlights the limitation of the connivence method since it uses that Enhanced (simplified) flow "under the hood" which limits permissions on the assumed role because of the intersection of multiple policies resulting in the least permissive one being applied to the credentials returned from STS.

Related ticket

…tityCredentials

This method uses the simplified authflow, which means users cannot explicitly set a session policy. This results in STS appending the Default Session Policy to the request, which can lead to limiting permissions even if they are explicitly attached to a role.
@RanVaknin RanVaknin requested a review from a team as a code owner June 22, 2023 22:33
@RanVaknin RanVaknin requested a review from kuhe June 22, 2023 22:33
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