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

Support for .aws/config? #53

Open
1 of 3 tasks
vspinu opened this issue Apr 11, 2020 · 2 comments
Open
1 of 3 tasks

Support for .aws/config? #53

vspinu opened this issue Apr 11, 2020 · 2 comments

Comments

@vspinu
Copy link

vspinu commented Apr 11, 2020

Please specify whether your issue is about:

  • a possible bug
  • a question about package functionality
  • a suggested code or documentation change, improvement to the code, or feature request

My .aws/config file has entries like this

[profile power]
role_arn = arn:aws:iam::1234567891011:role/PowerUserRole
source_profile = default
region = eu-central-1

Which specifies both region and role. This works with aws cli and, surely, there is a common expectation that it should work here as well. Currently .aws/config seems to be ignored (related #18, cloudyr/aws.s3#168).

Would be great if,

  1. profiles where searched in .aws./config first for region and role
  2. Any roles be assumed automatically like in aws cli
@jon-mago
Copy link
Contributor

This is absolutely something I would like to see working. Thank you for raising this ticket.

It's complicated by (as was noted over in cloudyr/aws.iam#13) a lot of the needed functionality being split over the two packages. I think to support this all of the credentials type code (parsing the config files, running down the list here, assuming a role as part of that) would be better living in one package, as otherwise they'd have circular dependencies anyway. It always seemed a little weird to me that aws.iam was both "make roles and users" and "assume roles", when the latter is another source of credentials. But ultimately, as long as which package a user needs to install is clearly communicated, I think that it doesn't matter which package this functionality ends up in.

@davidski
Copy link

davidski commented Oct 9, 2020

A reference to how the paws package is handling the credential_process problem - paws-r/paws@86de247, and an approach for source_profile as well - paws-r/paws#253 (comment). Would be fantastic to get similar support in the cloudyr ecosystem!

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

No branches or pull requests

3 participants