fffonion / xeHentai

Doujinshi downloader 绅士漫画下载
https://yooooo.us/2013/xehentai
GNU General Public License v3.0
729 stars 88 forks source link

一种图片下载失败的情况 #31

Closed xoxo1984 closed 7 years ago

xoxo1984 commented 7 years ago

使用时发现一种图片下载失败的情况,此时图片如此图所示,且大小也和网页上提示的大小不一致

建议增加图片大小检查功能 - 在下载图片前先记录图片大小,下载完成后检查实际大小是否和记录一致,不一致则重新下载

根据是否下载原图,可以有两种记录方式 原图 - 网页上显示"Download original 1200 x 1758 904.5 KB source"中的大小,注意:提示的文件大小是四舍五入的(实际大小926194字节) 非原图 - header中Content-Length的值

fffonion commented 7 years ago

试试新版? https://dl.yooooo.us/share/xeHentai/xeHentai-2.0.1.8.exe.zip

xoxo1984 commented 7 years ago

新版没有遇到过类似问题了(但我测试得少,不太确认) 另外,自己试了下才发现原图也能取Content-Length,之前的建议好蠢啊……

fffonion commented 7 years ago

content-length和实际下载如果不符的话requests应该是会报错的,所以既然下载到了残缺的图可能是

我觉得应该是前一种,你方便的话可以把log_level调到3看是否有"下载的图有猫饼",说明有第一种情况发生 On Fri, Nov 3, 2017 at 6:11 AM xoxo1984 notifications@github.com wrote:

新版没有遇到过类似问题了(但我测试得少,不太确认) 另外,自己试了下才发现原图也能取Content-Length,之前的建议好蠢啊……

— You are receiving this because you were assigned.

Reply to this email directly, view it on GitHub https://github.com/fffonion/xeHentai/issues/31#issuecomment-341663569, or mute the thread https://github.com/notifications/unsubscribe-auth/ACCVlS1G0LKnkvXaAIlcs9IqJ208F5PRks5syubZgaJpZM4QNxs0 .

xoxo1984 commented 7 years ago

突然想起来,图片“断裂”可能是由于超时造成 以前我程序里曾把线程里timeout设得很短,又没检查下载下来的图,很多图都是这样坏的 现在我用进程,且没设timeout,就没有遇到过图片断裂,直到我刚才某次手动中断

fffonion commented 7 years ago

我去配置里加个timeout的选项

fffonion commented 7 years ago

已更新https://dl.yooooo.us/share/xeHentai/xeHentai-2.0.1.8.exe.zip?171106 (因为和#32 重复了,timeout的后续问题,可以去#32 这里讨论