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

[Sankaku] "Expired link" image is being downloaded instead of some files #2958

Closed
pink-red opened this issue Sep 22, 2022 · 2 comments
Closed

Comments

@pink-red
Copy link
Contributor

This mostly happens when multiple videos are downloaded from a single search. Sankaku is very slow for me, and I believe that the extractor obtains multiple URLs at once in batches (pagination?). I believe that when the downloader is at the end of the batch, there is a high chance that all of the links are already expired.

Since the expired.png file is saved instead of these files, simply retrying the same command doesn't help, since gallery-dl thinks that the problematic files were already downloaded. Only manually finding and removing these files helps with this.

Is there a way to make gallery-dl detect expired links and refresh them before downloading, or at least to skip them, to simplify retrying the download?

@pink-red pink-red changed the title [Sankaku] "Expired link" image is being downloaded instead of some images [Sankaku] "Expired link" image is being downloaded instead of some files Sep 22, 2022
mikf added a commit that referenced this issue Sep 23, 2022
@mikf
Copy link
Owner

mikf commented Sep 23, 2022

gallery-dl grabs 100 posts per batch and those download links expire after 60 minutes, so you get expired.png files if it takes you longer than that to download them all.

I made it so it fails a download if it would download such a file (8506085). Updating expired links should also be possible, but that requires a bit more effort, I think, so went with the easier solution first ...

@mikf
Copy link
Owner

mikf commented Oct 1, 2022

The new release has a refresh option, which fetches new download links when the old ones have expired. (4089bce) It is not enabled by default since it's quite a lot of extra code, meaning it probably has a bug somewhere in it.

(This is one of the rare cases where a goto would have been helpful, by the way)

@mikf mikf closed this as completed Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants