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: Make progress file path respect --config-dir parameter #717

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

Bluetea577
Copy link
Contributor

问题

进度文件路径被硬编码为使用 const.ProgressPath,不会放在通过 --config-dir 参数指定的配置目录。相比之下,其他配置文件(token, settings等)都正确使用了指定目录。

修改内容

  • __init__ 方法中添加了 self._progresspath 的初始化
  • 更新所有进度文件操作,使用 self._progresspath 替代 const.ProgressPath

受影响的方法

  • _update_progress_entry()
  • _delete_progress_entry()
  • _upload_file_slices()

之前并发提交时遇到大文件会因为进度文件冲突报错,修改后再通过设置各自的config-dir就不会报错了。

## 问题
进度文件路径被硬编码为使用 `const.ProgressPath`,而没有遵循用户通过 `--config-dir` 参数指定的配置目录。相比之下,其他配置文件(token, settings等)都正确使用了指定目录。

## 修改内容
- 在 `__init__` 方法中添加了 `self._progresspath` 的初始化
- 更新所有进度文件操作,使用 `self._progresspath` 替代 `const.ProgressPath`
- 使用 `configdir + os.sep + filename` 的方式统一配置文件的路径拼接风格

## 受影响的方法
- `_update_progress_entry()`
- `_delete_progress_entry()`
- `_upload_file_slices()`

之前并发提交时遇到大文件会因为进度文件冲突报错,修改后设置各自的config-dir就不会报错了。
@houtianze houtianze merged commit fa2e87f into houtianze:master Jan 30, 2025
@houtianze
Copy link
Owner

感谢修正。新春快乐!

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