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

Switching accounts requires repeat login when using 2FA #13

Open
rdlugosz opened this issue Oct 17, 2017 · 5 comments
Open

Switching accounts requires repeat login when using 2FA #13

rdlugosz opened this issue Oct 17, 2017 · 5 comments

Comments

@rdlugosz
Copy link

rdlugosz commented Oct 17, 2017

When switching accounts (via heroku accounts:set foo), if the user has 2-factor authentication in use they will get a Invalid credentials provided message when they next issue a heroku command.

After entering email, password and the current 2FA code the user is able to use heroku as expected.

Note that the set command appears to execute successfully. The error doesn't happen until you try to run any other Heroku command.

@mobilutz
Copy link

mobilutz commented Feb 4, 2019

I think the big "problem" here is, that account:add only adds a username & password. But of course a 2FA accounts needs to validate these informations again and it looks like this validation is not saved by heroku-accounts.

@jughead
Copy link

jughead commented Feb 12, 2019

It looks like it now always requires the browser login even if 2FA is disabled: https://devcenter.heroku.com/changelog-items/1530?utm_source=dlvr.it&utm_medium=twitter

@alistairholt
Copy link

This is such a painful thing to deal with. I have to do a browser login every time I switch accounts and of course, the browser is guaranteed to already be logged into an account that isn't the one you are trying to authorise and the Heroku auth flow kind of sucks.

@kennethjiang
Copy link

The flow to change account has been broken since I upgraded Heroku cli to the version that uses browser to login. Really annoying and painful. Please get it fixed!

@JosephHalter
Copy link

+1

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

No branches or pull requests

6 participants