search

trek10inc / awsume

A utility for easily assuming AWS IAM roles from the command line.
https://awsu.me
MIT License
487 stars 90 forks source link

Specifying both role-arn and principal-arn as command-line arguments fails with TypeError #149

Closed Tantalon closed 11 months ago

Tantalon commented 3 years ago

If you specify both role-arn and principal-arn as command-line arguments, it fails with this:

  File "/usr/local/lib/python3.9/site-packages/awsume/awsumepy/app.py", line 132, in get_saml_credentials
    principal_plus_role_arn = ','.join(args.role_arn, args.principal_arn)
TypeError: str.join() takes exactly one argument (2 given)

This is the line of code, it's missing brackets: principal_plus_role_arn = ','.join(args.role_arn, args.principal_arn) should be: principal_plus_role_arn = ','.join([args.role_arn, args.principal_arn])