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

fix: 修复 pure-ftpd 上传缓慢问题 #6994

Merged
merged 2 commits into from
Nov 12, 2024
Merged

Conversation

bh1xaq
Copy link
Contributor

@bh1xaq bh1xaq commented Nov 10, 2024

What this PR does / why we need it?

pure-ftpd 上传缓慢是由于 login 过程中 Argon2id 算法的密码校验导致的,所以通过调整加密算法来处理 FTP 上传过程中的卡顿问题。

The slow upload in pure-ftpd is due to the Argon2id algorithm used for password verification during the login process. To address the lag in the FTP upload process, consider adjusting the encryption algorithm.

Summary of your change

调整了 pure-ftpd 加密算法,由 Argon2id 变更为 Bcrypt,已达到安全和速度的平衡。
影响范围:新增和修改用户密码,将不在使用原生 pure-pw 命令。其他操作不影响。

The encryption algorithm in pure-ftpd has been changed from Argon2id to Bcrypt, achieving a balance between security and speed.
Scope of impact: This change affects only the creation and modification of user passwords, which will no longer use the native pure-pw command. Other operations are unaffected.

Please indicate you've done the following:

  • [✅] Made sure tests are passing and test coverage is added if needed.
  • [✅] Made sure commit message follow the rule of Conventional Commits specification.
  • [✅] Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

@ssongliu ssongliu merged commit ee947b6 into 1Panel-dev:dev Nov 12, 2024
1 check was pending
liuruibin pushed a commit to liuruibin/1Panel that referenced this pull request Nov 12, 2024
* fix: 修复 pure-ftpd 上传缓慢问题

* fix: 修复 pure-ftpd 修改密码用户名的判断依据

---------

Co-authored-by: BH1XAQ <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants