search

nextcloud / server

☁️ Nextcloud server, a safe home for all your data
https://nextcloud.com
GNU Affero General Public License v3.0
26.53k stars 3.99k forks source link

Show status of 2FA usage in user management #13395

Open j-ed opened 5 years ago

j-ed commented 5 years ago

Expected behavior

The 2FS support has greatly been improved with NC 15.x, but it is currently not possible for an administrator to check if a user has enabled any kind of 2FA for his account. Best would be if this information could be seen at a central point, like e.g. user management.

Such an indicator could e.g. look like this:

20190106_nc_user_management-2

The color of the icon could e.g. indicate the status of a 2FA activation, red- not enabled/activated, yellow - activated but not yet used, green - everything is ok.

By clicking on the icon the enabled 2FA apps could be shown, like twofactor_gateway: telegram, Two factor U2F or twofactor_nextcloud_notification etc.

Environment

Server Configuration

OS: Linux 3.16.50 Web server: Apache2 2.4.37 Database: MariaDB 10.2.19 PHP version: 7.2.13 Nextcloud version: 15.0.0

Client Configuration

Browser: Mozilla Firefox 64.0 Operating system: Windows 10

MorrisJobke commented 5 years ago

cc @ChristophWurst

ChristophWurst commented 5 years ago

Should be doable with the stateful providers data.

TecJon commented 5 years ago

Good idea @j-ed. However, I find the proposed colors rather flashy. How about a gray icon if it is used (maybe add a small check to the icon) and no icon if not enabled/not used?

j-ed commented 5 years ago

@TecJon The last word has always the NC design team ;-) Unfortunately it is not possible to show 3 different states with an on/off logic. But let's see what other ideas are coming up on this.

TecJon commented 5 years ago

That’s true, my proposed icon would only distinguish between enabled & used and not used (in this case it’s not visible). Is the state “enabled but not used” important information for the general NC admin? I think the colors of the icon would need explanation. The 2FA icon with a check could be self-explanatory. Additionally, I believe the UI should not be cluttered with colors where it’s not absolutely necessary. You’re right, it’s for the design team to decide. I’m just adding my thoughts. Thank you for your good idea. 👍🏻

On 8. Jan 2019, at 13:42, j-ed notifications@github.com wrote:

@TecJon The last word has always the NC design team ;-) Unfortunately it is not possible to show 3 different states with an on/off logic. But let's see what other ideas are coming up on this.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

szaimen commented 3 years ago

It seems like this is still valid.

j-ed commented 3 years ago

Yes, that issue is still valid.