pfaffman
(Jay Pfaffman)
28.Февраль.2023 19:55:10
1
Если у пользователя есть ключи безопасности, они не удаляются, поэтому пользователь всё ещё не может войти в систему.
Если только UserSecondFactor каким-то образом не сломан, что кажется маловероятным, ключи безопасности также необходимо удалить, примерно так:
UserSecurityKey.where(user_id: user.id).destroy_all
2 лайка
sam
(Sam Saffron)
28.Февраль.2023 23:30:09
2
Действительно, это выглядит как разумное улучшение задачи. Можешь отправить PR?
1 лайк
pfaffman
(Jay Pfaffman)
01.Март.2023 00:01:53
3
Если бы это была всего одна строка, я бы уже сделал это, но мне действительно нужно создать один или два теста, что звучит гораздо страшнее (и вчетверо больше работы!)
Но, возможно, я смогу проявить взрослость.
Просто один тест, который создаёт ключ 2FA и затем проверяет, что он был удалён? И, полагаю, в коде 2FA где-то есть возможность создать тестовый ключ?
2 лайка
pfaffman
(Jay Pfaffman)
08.Март.2023 14:57:23
4
Похоже, что для этой задачи по взятию задания нет никаких спецификаций, поэтому есть только одна строка:
main ← pfaffman:rake-2fa-security-key
merged 08:38PM - 09 Mar 23 UTC
https://meta.discourse.org/t/task-users-disable-2fa-does-not-disable-security-ke… ys/256547
I thought I'd need to add a test, but I don't see any tests for this rake task, and the change is pretty straightforward, so maybe this single line is all that's needed.
1 лайк
Эта тема была автоматически закрыта через 4 дня. Новые ответы больше не принимаются.