Skip to content

Add AssumeRole functionality for credentials/config files #1025

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

Closed
wants to merge 1 commit into from

Conversation

davelantos
Copy link

The role_arn and source_profile fields within a credentials file have been
ignored up until now. This fix adds support for assuming a role from a
credentials and/or config file.

@davelantos
Copy link
Author

Can someone from the AWS C++ sdk team suggest how to unit test this?
In order to unit test this, we need a role to assume. How do you generally handle creating various AWS resources for unit testing?

The role_arn and source_profile fields within a credentials file have been
ignored up until now.  This fix adds support for assuming a role from a
credentials and/or config file.
@davelantos davelantos force-pushed the configFileAssumeRole branch from b222218 to 1bec9a2 Compare December 3, 2018 21:55
@davelantos
Copy link
Author

Added some basic unit tests. I guess I don't need to actually assume a role, I can just mock the STS role functionality.

@ltn100
Copy link
Contributor

ltn100 commented Jul 4, 2019

Any chance you could resolve these conflicts, @LantosDavid? I would love to get this feature!

@singku
Copy link
Contributor

singku commented Sep 9, 2019

This feature has been supported by SDK team, see 3dce455

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