Repository navigation
feat:下载视频与音频,日志同步写入磁盘 - #24
Conversation
|
能提供一些对应作者用于测试吗 |
我正在爬取的创作者是https://www.ifdian.net/a/miaowasmr 确实没有考虑作者使用临时链接和外链的情况，不清楚遇到时会有什么行为 视频和音频的链接来源于响应中的data->post->audio和video字段，这里应该不会出现这种情况？ |
|
主体功能没什么问题，就是上面的几个细节需要讨论一下。还需要补充一下README中的功能介绍 |
|
我进行了一些对应的更新,为音视频的下载增加了命令行参数进行开关控制.同时提供了一个下载失败时跳过而不是终止程序的开关来规避之前讨论提到的问题.readme我也改了一下. |
恕我眼拙，最新的commit里我只看到了命令行参数控制下载行为的新增功能，并没有看到您对我之前给出的修改建议的回应？🤔 |
很抱歉我的表述有问题,我的意思我提供的控制下载行为的功能是为了规避: "我担心通过这种方式插入的链接是临时链接，后续无法访问"这一句提到的问题,如果您是指提供对应作者进行测试的话,很抱歉我只发电了上面提到的作者,不过我已经对该作者爬取了约270G资源,未见明显异常. |
|
我的意思是我想知道您对我之前code review的建议有何看法，像“默认持久化日志”这个功能我认为是需要讨论一下的 |
|
关于默认日志持久化我认为是必要的,与只下载文章内容和图片不同,音视频的下载时间过长,保留一个可供翻阅的日志对于查找下载失败的文章并手动下载补齐而言更加方便. |
|
不过默认日志持久化的原因是下载音视频本来也是默认开启的,现在改为不默认启用的话,确实改为不默认启用更加合适一些. |
|
ok，那我就merge之后移除掉日志持久化，剩下的几个小问题按我的思路改了 |
|
好的,按您的思路进行修改就行,感谢您进行的review和merge |
2 participants
支持下载视频和音频,会在md文件中插入html标签引用.日志持久化方便查找失败的下载