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

Add missing permission to assume Cross-Account-Org R/O role #509

Merged
merged 2 commits into from
Aug 31, 2022

Conversation

sbkok
Copy link
Collaborator

@sbkok sbkok commented Aug 31, 2022

Why?

The Generate Pipeline Input lambda function tried to assume into the cross-account access role, to read the organization API.
This was not allowed, as it was not permitted by the cross-account access role as a trusted role that can assume into the read-only role in the management account.

What?

Addressed by adding a specific role name for the role that is used by the Generate Input lambda function. Plus putting that role on the allowed list of roles that are allowed to assume the cross-account access role for read-only access to the organization API.


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

sbkok added 2 commits August 31, 2022 09:28
**Why?**

The Generate Pipeline Input lambda function tried to assume into the
cross-account access role, to read the organization API.
This was not allowed, as it was not permitted by the cross-account access role
as a trusted role that can assume into the read-only role in the management
account.

**What?**

Addressed by adding a specific role name for the role that is used by the
Generate Input lambda function. Plus putting that role on the allowed list of
roles that are allowed to assume the cross-account access role for read-only
access to the organization API.
@sbkok sbkok added the bug Something isn't working label Aug 31, 2022
@sbkok sbkok added this to the v3.2.0 milestone Aug 31, 2022
@sbkok sbkok requested review from javydekoning and StewartW August 31, 2022 07:30
@sbkok sbkok merged commit ae52f13 into awslabs:master Aug 31, 2022
@sbkok sbkok deleted the fix/cross-account-org-ro-pipeline-usage branch August 31, 2022 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants