list output is too simple

[ozbillwang]

Currently the list output is too simple, I use it to set a default profile, but sometime I need switch to other account, which it doesn't make sense for me, if always list as default.

can we have output with additional information, such as profile name, aws account alias and adfs_config.role_arn?

[venth]

Would you like to paste a mockup (could be formatted text), so it would be easier for me to see the results of the change?

[ozbillwang]

change from:

$ aws-adfs list
Available profiles:
    * default

to something like this, so I can easily know what's the current default profile.

$ aws-adfs list
Available profiles:
    * default | adfs-xxx-nonprod | arn:aws:iam::xxxxxxxx:role/administrator

If getting aws account alias is not possible, because you need assume role to get the account detail, the role_arn will be useful, which can get directly from ~/.aws/config

[ozbillwang]

pull request has been raised.

[ozbillwang]

PR has been merged.

[venth]

Released with version 0.2.2

[ozbillwang]

@venth

Saw the codes you updated in aws_adfs/list_profiles.py, seems line 19 is not required any more.

https://github.com/venth/aws-adfs/blob/master/aws_adfs/list_profiles.py#L19

[venth]

You're right 😄I'll fix it. Thanks.