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

Password confirmation when changing second factor configuration #5986

Open
6 of 20 tasks
bedhub opened this issue Oct 13, 2023 · 1 comment · May be fixed by #7933
Open
6 of 20 tasks

Password confirmation when changing second factor configuration #5986

bedhub opened this issue Oct 13, 2023 · 1 comment · May be fixed by #7933
Assignees

Comments

@bedhub
Copy link
Contributor

bedhub commented Oct 13, 2023

As a user I want to make sure that I cannot be locked out of my account or lose second factor protection by someone who has access to my logged in account.

AC

  • Before configuring second factor the user is presented with a password challenge
  • When deleting a second factor the user is also presented with a password challenge
  • The server enforces the password validity for 2FA changes
  • Old clients still allow the changes without the password confirmation
  • For admin users changing second factors of other admin users the password of the active admin is verified instead of the user password
  • For admins changing second factors of regular (non-admin) users there is no password check

Notes

Ask for the password before starting 2FA flow

Tasks

  • Write down the design for token system
  • Add a service for requesting account token/admin token
  • Handle admin users changing 2fa for other users (admin and non-admin)
  • Add password request form upon changing second factors
  • Implement a check for the token validity (deactivated for now)
  • Handle expired tokens on the client (an error message)

Estimated time: 24h
taken: 25.75h

Test Notes

  • With Current Version
    • be able to add 2fa
    • be able to delete 2fa
    • take a long time, request is denied
      - steps: enter password, keep the 2fa setup dialog open for longer than 30 min, try to finish setup, the request should be denied
    • enter wrong password, request is denied
      • when adding
      • when deleting
    • managing users
      • when adding/removing 2fa from non-admin user, there is no prompt for a password
      • when adding/removing 2fa from another admin user, this is a prompt for a password
      • try to add/remove 2fa from a deactivated user, a pop-up saying that the user is deactivated appears
  • With older version
    • be able to add 2fa
    • be able to delete 2fa
@wrdhub wrdhub assigned wrdhub and BijinDev and unassigned BijinDev Nov 6, 2024
wrdhub added a commit that referenced this issue Nov 12, 2024
wrdhub added a commit that referenced this issue Nov 13, 2024
wrdhub added a commit that referenced this issue Nov 13, 2024
wrdhub added a commit that referenced this issue Nov 13, 2024
wrdhub added a commit that referenced this issue Nov 13, 2024
wrdhub added a commit that referenced this issue Nov 14, 2024
wrdhub added a commit that referenced this issue Nov 15, 2024
hrb-hub pushed a commit that referenced this issue Nov 19, 2024
@charlag
Copy link
Contributor

charlag commented Nov 20, 2024

Might be a good idea to merge it after labels and not put it into the same release.

wrdhub added a commit that referenced this issue Nov 25, 2024
wrdhub added a commit that referenced this issue Nov 28, 2024
@wrdhub wrdhub added this to the Next client release milestone Nov 28, 2024
wrdhub added a commit that referenced this issue Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants