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.

Sign up for GitHub

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

Jump to bottom

新增强制全局2FA开关 #1490

Merged
merged 1 commit into from
May 7, 2022
Merged

新增强制全局2FA开关 #1490

merged 1 commit into from
May 7, 2022

Conversation

nick2wang
Copy link
Collaborator

根据@liujing1087 和@thcoffee 的建议，新增全局配置项ENFORCE_2FA，可强制全员开启2FA，如果用户未配置2FA，则在登录验证后引导开启

配置项：
image

未配置2FA用户登录时引导开启：
image

@thcoffee
Copy link
Contributor

thcoffee commented May 6, 2022

昨天还觉得挺遗憾呢，今天就修复了。very good

@hhyo hhyo merged commit 6de47f7 into hhyo:master May 7, 2022
@nick2wang nick2wang deleted the enforce-2fa branch May 7, 2022 02:12
@nick2wang nick2wang mentioned this pull request May 7, 2022
Merged
@nick2wang nick2wang mentioned this pull request Jun 14, 2022
Merged
@FSerKevin
Copy link

扫描2FA二维码后 谷歌认证器上显示是Archery 请问Archery字样是否可以更改，在哪里更改

@nick2wang
Copy link
Collaborator Author

扫描2FA二维码后 谷歌认证器上显示是Archery 请问Archery字样是否可以更改，在哪里更改

直接修改代码中的issuer_name

Archery/common/twofa/totp.py

Lines 81 to 84 in 287e74a

# 生成二维码
qr_data = pyotp.totp.TOTP(secret_key).provisioning_uri(
username, issuer_name="Archery"
)

@FSerKevin
Copy link

谢谢@nick2wang 解决了我的问题

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hhyo hhyo hhyo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nick2wang @thcoffee @FSerKevin @hhyo