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

关闭密码复杂度校验 #937

Closed
NyaMisty opened this issue Dec 12, 2023 · 6 comments
Closed

关闭密码复杂度校验 #937

NyaMisty opened this issue Dec 12, 2023 · 6 comments
Labels
bug Something isn't working

Comments

@NyaMisty
Copy link

NyaMisty commented Dec 12, 2023 •
edited
Loading

问题描述

我不觉得一个self-hosted的程序应该强制设置最低的密码复杂度。
@NyaMisty NyaMisty added the bug Something isn't working label Dec 12, 2023
@jeessy2
Copy link
Owner

jeessy2 commented Dec 12, 2023

有些需要暴露在公网的，还有些内网也不是很安全。
密码复杂度可以降低点，暂时不考虑关

@NyaMisty
Copy link
Author

用户布在哪里其实是他自己的选择，跟你这个软件没有一丁点的关系。
只能说你这样操作让人感觉很难受。
如果说在这里协商失败那我其实也没啥办法，我会自己fork一份关闭掉它的repo。

@jeessy2
Copy link
Owner

jeessy2 commented Dec 12, 2023

Fork吧。对你来说可能麻烦点，有些人设置123456暴露公网，然后被破解了就完了

@NyaMisty
Copy link
Author

我仔细看了下，发现核心的问题是这个patch里面提到的known issue并没有被修复。。。。

#779

难怪我手动修改配置都没法保存。其实这个弄好我就可以直接改配置文件里面的密码了。

@jeessy2
Copy link
Owner

jeessy2 commented Dec 19, 2023

我仔细看了下，发现核心的问题是这个patch里面提到的known issue并没有被修复。。。。

#779

难怪我手动修改配置都没法保存。其实这个弄好我就可以直接改配置文件里面的密码了。

感兴趣的话，欢迎PR

@jeessy2 jeessy2 mentioned this issue Jan 10, 2024
Merged
@jeessy2
Copy link
Owner

jeessy2 commented Jan 11, 2024 •
edited
Loading

v6.0.0在未开启公网的条件下，已经减低密码复杂度的要求

@jeessy2 jeessy2 closed this as completed Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@NyaMisty @jeessy2