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(ftp-server): large transfer leads to client timeout #7639

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

KirCute
Copy link
Contributor

@KirCute KirCute commented Dec 10, 2024

刚刚pr合并的时候我还在测,结果真的就发现了两个bug,哭了。

问题1:上传大文件结束时,服务端会阻塞式地将本地的临时文件上传到网盘,期间FTP客户端会显示100%不动。这本来是预期的行为,但某些FTP客户端在这种情况下会认为上传超时失败,所以只能改成不阻塞的“以任务形式上传”

问题2:临时文件不会自动删除,而且挂载另一个AListV3时没法上传,发现是因为临时文件在SetTmpFile以后还得手动加入Closers里面,否则文件不能自动关闭。

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.

1 participant