Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ 新功能:2FA (Email & TOTP) #274

Open
wants to merge 21 commits into
base: develop
Choose a base branch
from

Conversation

cfdxkk
Copy link
Member

@cfdxkk cfdxkk commented Nov 4, 2024

✨ 新功能:2FA (Email & TOTP)
合并目前为止写完的两步验证功能。
@otomad @Aira-Sakuranomiya

⚠ 强依赖于某些后端 API,请使用本地后端的 develop 分支测试
🚧 在充分测试前禁止合并
🏗 很快就有一个与生产环境 1:1 的线上测试后端,我还在跟 vultr 申请配额

测试焦点

  • 两步验证(设置 -> 安全)
    • 邮箱两步验证
      • 邮件发送及冷却时间
    • TOTP 两步验证
      • 二维码能否正确生成
      • 下载备份码和恢复码的功能(请重点测试 macOS 及 PE 端是否正确工作)
    • 关闭两步验证
  • 登录
    • 使用 / 不使用 2FA 的登录流程
    • 在使用 TOTP 二重验证时能否使用备份码及恢复码登录
  • 如果没有开启 2FA,首页会出现警告(是否实装该功能 ❓)

已知问题

  • 多选框在 SSR 时工作正常,在普通的路由切换时会出现闪烁(默认选中第一个,然后切换为正确的选项)

未包含功能

以下功能未能包含在本次更新中

  • 设置 -> 安全 页面中的 “密码更新时间” 仍然是占位符

feat(2FA): update login window and security page (work in progress)
@cfdxkk cfdxkk added the enhancement New feature or request label Nov 4, 2024
@cfdxkk cfdxkk self-assigned this Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants