nextcloud/server

Clear / refresh 2FA backup codes

Aperta

#9036 aperta il 30 mar 2018

Vedi su GitHubGuida contributor
 (9 commenti) (0 reazioni) (0 assegnatari)PHP (4865 fork)batch import
1. to developenhancementfeature: authenticationgood first issuehelp wanted

Metriche repository

Star
 (34.953 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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...

Guida contributor

Direzione di ricerca
Esamina l'app twofactor totp (apps/twofactor totp/) e il suo schema del database. Rivedi la discussione precedente nell'issue #244. Implementa la logica di pulizia per eliminare i codici di backup quando il 2FA viene disabilitato, e rimuovi anche i codici per gli utenti eliminati. Controlla gli event hook e i file di migrazione esistenti.
Tech stack
php
Dominio
backendsecurity
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PHPNextcloud development environmentMySQL/PostgreSQL
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70