사용자가 보안 키를 보유하고 있는 경우, 해당 키가 삭제되지 않아 사용자는 여전히 로그인할 수 없습니다.
UserSecondFactor가 어딘가에서 깨져 있지 않는 한(가능성이 낮아 보이지만), 보안 키도 삭제되어야 합니다. 다음과 같은 코드가 필요합니다:
UserSecurityKey.where(user_id: user.id).destroy_all
2개의 좋아요
작업에 대한 합리적인 개선인 것 같은데, PR을 보내줄 수 있을까?
1개의 좋아요
한 줄이면 이미 처리했을 텐데, 실제로는 테스트를 하나둘 만들어야 해서 좀 더 부담스럽고 (그리고 4배의 작업량!)
하지만 내가 좀 더 어른스럽게 행동해 볼 수도 있겠지.
2FA 키를 생성한 후 그 키가 제대로 삭제되는지 확인하는 테스트 하나만 만들어도 될까? 그리고 2FA 코드 어딘가에 더미 키를 생성하는 커피(coffee)가 필요한 것 같은데?
2개의 좋아요
글쎄요, 이 작업에 대한 명세서가 있는 것 같지 않으므로, 단순히 한 줄입니다:
1개의 좋아요
이 주제는 4일 후 자동으로 닫혔습니다. 새로운 답글을 더 이상 작성할 수 없습니다.