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

Add bare output mode, Enable ~/.passcode usage #11

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ibexa-yuna
Copy link

This PR contains code from @rirze to enable bare output and enable usage of ~/.passcode file,
alongside my changes to exit after the first code is output, so that app can be used in conjunction with cli tools, like pbcopy on OSX.

Fixes #9

@rirze
Copy link

rirze commented Mar 26, 2021

... I really didn't expect someone to pick up my fork and bring it back upstream. I understand your use case, as it's probably the same as mine, but storing the password in a plaintext file is very unsafe from a security perspective.

If you really want to merge this back into upstream, try to figure out a way to make this more secure.

@ibexa-yuna
Copy link
Author

I do know it defeats the purpose of MFA, as you are literally putting all eggs in one basket.
The thing is, with MFA enabled AWS does not provide any UI to authenticate from CLI in a "human-friendly way".

What I do is I link authenticator with aws-mfa and I have the luxury to do this, as AWS-enabled machine is dedicated and in a sterile environment. But not many people do...

I would really like to hear the feedback fro @JeNeSuisPasDave on this topic.

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.

Missing ability to generate 1 code, for automation purposes
2 participants