nextcloud/server

Clear / refresh 2FA backup codes

Open

#9.036 aberto em 30 de mar. de 2018

Ver no GitHub
 (9 comments) (0 reactions) (0 assignees)PHP (4.865 forks)batch import
1. to developenhancementfeature: authenticationgood first issuehelp wanted

Métricas do repositório

Stars
 (34.953 stars)
Métricas de merge de PR
 (Mesclagem média 20d) (627 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Analise o aplicativo twofactor totp (apps/twofactor totp/) e seu esquema de banco de dados. Revise a discussão anterior na issue #244. Implemente a lógica de limpeza para excluir os códigos de backup quando o 2FA for desativado, e também remova os códigos de usuários excluídos. Verifique os event hooks e os arquivos de migração existentes.
Pilha de tecnologia
php
Domain
backendsecurity
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PHPNextcloud development environmentMySQL/PostgreSQL
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70