nextcloud/server

Clear / refresh 2FA backup codes

Open

#9 036 ouverte le 30 mars 2018

Voir sur GitHub
 (9 commentaires) (0 réactions) (0 assignés)PHP (4 865 forks)batch import
1. to developenhancementfeature: authenticationgood first issuehelp wanted

Métriques du dépôt

Stars
 (34 953 stars)
Métriques de merge PR
 (Merge moyen 20j) (627 PRs mergées en 30 j)

Description

as already mentioned in https://github.com/nextcloud/twofactor_totp/issues/244, maybe just a question... but shouldn't the Backup-Codes be cleared/deleted after an user disables his 2FA?

in the database they are still present, also for users which were completely deleted ages ago.

i'm not sure if this may even become a security issue, especially if a user enables 2FA again...

Guide contributeur

Direction de recherche
Examinez l'application twofactor totp (apps/twofactor totp/) et son schéma de base de données. Consultez la discussion précédente dans l'issue #244. Implémentez une logique de nettoyage pour supprimer les codes de secours lorsque l'authentification à deux facteurs est désactivée, et supprimez également les codes des utilisateurs supprimés. Vérifiez les hooks d'événements et les fichiers de migration existants.
Stack technique
php
Domaine
backendsecurity
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.
Claire
Prérequis
PHPNextcloud development environmentMySQL/PostgreSQL
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70