-
Notifications
You must be signed in to change notification settings - Fork 257
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
Two-factor authentication (2FA) support #359
Comments
There is a hint in the documentation w.r.t. missing 2fa: http://doc.wallabag.org/en/master/user/android.html#two-factor-authentication-2fa |
I just bumped into this, it would be cool to support. A better error message would be helpful too :) |
until #359 is implemented, the user gets a concretized error message.
The thing is the app currently uses two ways to communicate with the server:
Implementing 2FA support would require to further complicate the first part. The problem is that the app shouldn't be doing that part at all: there should be a different way of getting API credentials. So implementing 2FA at this point would be like creating a crutch for a crutch. My memory is somewhat hazy on the matter, but as a workaround one probably can disable 2FA for the first setup, then after confirming that the app functions ok, enable 2FA again. |
Hey @di72nn , Thanks for the detailed response - this makes a lot of sense. A secondary thought then; would it be possible for the app to instead prompt for API credentials (that the user can create via the web UI)? It may not be as great of an experience compared to simply logging in and entering the 2FA, but it might be a good-ish middle ground for the time being. In the mean time, your workaround worked great for me! Thank you! The app is great! :) Best, |
In the first start wizard is no way to set up with client ID and secret. i always get an error. |
@tasmo you don't need to go through the wizard - just leave it (by pressing "back" or "skip") and manually set up the credentials in settings. |
The app does not support 2FA (not relevant for wallabag v1.9).
Also maybe add a notice somewhere about 2FA not being supported in the meantime.
The text was updated successfully, but these errors were encountered: