Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

添加下载结果校验 #127

Open
unqqunqq opened this issue Aug 27, 2023 · 2 comments
Open

添加下载结果校验 #127

unqqunqq opened this issue Aug 27, 2023 · 2 comments

Comments

@unqqunqq
Copy link

通过docker部署了最新的代码,很好用,感谢。
使用中发现有大概5%左右的文件下载时没下全,音乐只有一半,文件大小也跟查询结果有出入。手动删除本地文件重新下载后是正常的。
手动逐轨检查确实耗时较久,不知能否增加下载结果的简单校验功能?

@QiuChenly
Copy link
Collaborator

无法校验,我说下原因。

  1. qq音乐确实能知道文件实际大小是多大,但是我一开始做了检查,后面因为要增加元数据 歌词 封面图片 会增大文件体积,所以天然就不可能匹配qq服务器记录的大小。
  2. 如果文件只有一半 可能是线程数太高了,你的带宽比较低导致丢包,建议降低线程数。

@qupig
Copy link

qupig commented Sep 7, 2023

确实遇到不少下载文件存在错误或明显不全的情况。

下载校验还是有必要做的。

已下载文件校验大小确实如 @QiuChenly 所说,可能需要剥离元数据再计算,稍复杂。

看能否在写入前进行大小检查。

另元数据中本身包含的 MD5 of the unencoded content 也应该可以用作数据校验。

但我暂时没查到该元数据的定义和计算方法。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants