Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Prompt for 2fa on stderr instead of stdout #1234

Merged
merged 1 commit into from Oct 17, 2014

Conversation

msakrejda
Copy link
Contributor

This means that a 2fa prompt is still visible if, e.g., one is piping the result of a heroku command which requires 2fa into another command:

Before this patch:

maciek@gamera:~/code/heroku$ bin/heroku config -a db-corral | grep foo
# hangs indefinitely after the above; it does actually work if i just feed it
# my 2fa code blind, but i have to know to do that
ccccccdlnndrcidhrtgligivrvvlfetgdnkukhrrhfef 
foo:                               4
test:                              foo
maciek@gamera:~/code/heroku$

After this patch:

maciek@gamera:~/code/heroku$ bin/heroku config -a db-corral | grep foo
# now shows prompt before waiting for 2fa code
Two-factor code: ccccccdlnndrtrnebkkclicdbbfcrhiideekjgncetun
foo:                               4
test:                              foo
maciek@gamera:~/code/heroku$

This means that a 2fa prompt is still visible if, e.g., one is
piping the result of a heroku command which requires 2fa into
another command.
jdx pushed a commit that referenced this pull request Oct 17, 2014
Prompt for 2fa on stderr instead of stdout
@jdx jdx merged commit a8e624f into heroku:master Oct 17, 2014
@msakrejda
Copy link
Contributor Author

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants