warp-tech/warpgate

[BUG] Login asks for OTP key when none is associated with the account.

Ouverte

#981 ouverte le 16 avr. 2024

Voir sur GitHub
 (1 commentaire) (0 réaction) (0 personne assignée)Rust (281 forks)github user discovery
area/uihelp wanted

Métriques du dépôt

Stars
 (7 061 étoiles)
Métriques de merge PR
 (Merge moyen 3j 5h) (70 PRs mergées en 30 j)

Description

I added an OTP key to a user then realised I couldn't add it (Bitwarden wouldn't scan the QR code and it didn't display the raw hash).

I then deleted the OTP key from the account and clicked update. If I check in the admin panel there is no OTP key associated with the account, but when I want to log in with that account in the web interface, it asks for an OTP value.

Reproduction:

  1. Log in as admin
  2. Add otp key to account
  3. Click next (why is there no confirmation for the OTP key?)
  4. Save the account, but stay on the page
  5. Delete the OTP key
  6. Save the account again
  7. Log out as admin
  8. Try to log in as the changed user. Warpgate asks for an OTP key after password, which it shouldn't

I'm on version v0.9.1

Guide contributeur

Direction de recherche
Examinez la logique de validation OTP dans le gestionnaire de connexion pour comprendre pourquoi il demande encore un OTP après la suppression de la clé. Vérifiez si la mise à jour de la base de données n'est pas propagée à la session ou si l'exigence OTP est mise en cache.
Stack technique
rust
Domaine
backendsecurityauthentication
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70