No credential exports printed when using docker.

[jtszalay]

The credential exports aren't printed when using docker. It seems the config.profile has a value no matter what and the else that should print them is never reached.

[wcharaka]

@jgissend10 I think you are right. It seems in Configuration the self.profile is defaults to sts hence

if config.profile:
        config.write(amazon_client)
else:
        amazon_client.print_export_line()

will always be true so it would never print. two options I can think of is either you can print it always. or you print based on a parameter. I find the latter more secure as I m not sure if we should just print the keys in the terminal. Thoughts..?

[jtszalay]

I'd prefer a parameter and to just replace the else with 'if parameter:' that way I can print it out even if we're writing the config to the client.

…

On Thu, Aug 16, 2018, 04:17 wcharaka ***@***.***> wrote: @jgissend10 <https://github.com/jgissend10> I think you are right. It seems in Configuration the self.profile is defaults to sts hence if config.profile: config.write(amazon_client) else: amazon_client.print_export_line() will always be true so it would never be true. two options I can think of is either you can print it always. or you print based on a parameter. I find the latter more secure as I m not sure if we should just print the keys in the terminal. Thoughts..? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#96 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAj-vqxTL3iZWgk-qC9-1S6423u3hT0Tks5uRSqYgaJpZM4V4u5n> .

[stevemac007]

Agree, lets add an option to print the credentials so you have the option to use this no matter which way you use it.

[jtszalay]

Any update on this?

[stevemac007]

@wcharaka are you still looking at this one, otherwise I'm happy to pick this up and get it working agin.

[wcharaka]

@stevemac007 I m on it..

[wcharaka]

This has been fixed and released.