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

TOTP authenticator UI strings only mention "Google Authenticator" specifically, instead of being generic #946

Open
nekohayo opened this issue Oct 30, 2024 · 2 comments

Comments

@nekohayo
Copy link

Based on this thread and what I see on my personal server running Virtualmin 7.20.2 + Webmin 2.202 + Usermin 2.102 from your Debian 12 repositories, it seems that VirtualMin, Webmin, Usermin have support for TOTP 2FA; if I am not mistaken, that support is generic, independent of which exact application is used, yes?

If so, it would probably be good to do some grepping / search-and-replace across all three codebases to make it generic and not only mention Google Authenticator specifically. There are plenty of other apps out there that your users might want to use, but currently when looking at the UI I can't be sure that I can, because it says "Google Authenticator" specifically. I mean, I have a guess, but I'm not sure, and less geeky users may not know at all. In addition to linking to a neutral list of applications, it could be fine to mention Google Authenticator as an example in a (?) help tooltip for convenience, just not present it as the only thing?


This is the same logic/recommendation as I have made in Tiki.

@iliajie
Copy link
Collaborator

iliajie commented Oct 30, 2024

Hello Jeff!

Thanks for the heads up! Could you try nightly Webmin version and see if the problem is solved?

@nekohayo
Copy link
Author

Oh, I did not expect that you were already independently fixing this in the dev version :)
Unfortunately I don't think I have an easy way to test that here. Do you have an idea of approximately when this might reach the stable releases of those three components? If so I can make myself a reminder note to retest and come back to close this ticket at that time.

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

2 participants