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 file renaming after task completion #104

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nasbdh9
Copy link
Contributor

@nasbdh9 nasbdh9 commented Nov 8, 2024

fix: #103

应用补丁后下载任务完成后的

扩展名重命名(根据下载图片url中的扩展名,比如目前新上传的文件处理后的非原图都应该是webp,之前上传的文件处理的文件非原图则是jpg)

都应该是正常的,并且任务完成后能生成.xehdone

@fffonion
Copy link
Owner

fffonion commented Nov 8, 2024

没看懂,改了啥 🤔

@nasbdh9
Copy link
Contributor Author

nasbdh9 commented Nov 8, 2024

现在的扩展名重命名是有问题的(应该是说这个问题一直都有),就修这个,以前后台压缩非原图都是jpg所以大家都不太发现这个问题?如果不修这个问题的话现在下原图的扩展名也是全部变webp的
image

@fffonion
Copy link
Owner

fffonion commented Nov 8, 2024

下载过程中所有图片都会用id.jpg,是因为save_tasks=False的时候也可以续传。
然后只能用网页底下写的文件名当成原始文件名,因为有些url里没有文件名

@fffonion
Copy link
Owner

fffonion commented Nov 8, 2024

原始文件名是从这里取出来的 https://github.com/fffonion/xeHentai/blob/master/xeHentai/filters.py#L194
如果有问题应该是这里的问题,可能正则需要改一下

@nasbdh9
Copy link
Contributor Author

nasbdh9 commented Nov 8, 2024

因为有些url里没有文件名

目前好像没这种情况了

原始文件名是从这里取出来的 https://github.com/fffonion/xeHentai/blob/master/xeHentai/filters.py#L194 如果有问题应该是这里的问题,可能正则需要改一下

我观察是在完成任务后的def rename_fname才出的问题,所以才这么改

@fffonion
Copy link
Owner

fffonion commented Nov 8, 2024

目前好像没这种情况了

我记得和本子的热门程度有关,有些是走pcdn的,有些不是,会影响url的pattern

我观察是在完成任务后的def rename_fname才出的问题,所以才这么改

可以看一下renamed_map里面的内容正不正确,因为你这个PR删掉的内容都是需要的(可以blame看当时加上的原因),所以我有点困惑到底什么地方有bug

@nasbdh9
Copy link
Contributor Author

nasbdh9 commented Nov 8, 2024

可以看一下renamed_map里面的内容正不正确,因为你这个PR删掉的内容都是需要的(可以blame看当时加上的原因),所以我有点困惑到底什么地方有bug

改完的话下载也是按照id.jpg的,完成后修改扩展名和根据rename_ori修改文件名
我也想不到原先的renamed_map里有啥问题(逃

@nasbdh9
Copy link
Contributor Author

nasbdh9 commented Nov 8, 2024

我铸币了,把duplicate_map也删了,现在补回去了(

@nasbdh9
Copy link
Contributor Author

nasbdh9 commented Nov 8, 2024

跑了几个有重复文件的图廊,比如:https://exhentai.org/g/2974767/b343ca5dfa/
现在就剩下duplicate_map复制出来的文件没法修改正确的文件扩展名,大佬有头绪吗@fffonion
应该都没问题了
还是有问题duplicate出来的文件有部分没有重命名扩展名

@fffonion
Copy link
Owner

fffonion commented Nov 10, 2024

等我有空的时候研究一下!

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