ccloli / E-Hentai-Downloader

Download E-Hentai archive as zip file
GNU General Public License v3.0
1.84k stars 138 forks source link

Won't retry, spits out Failed! (Network Error) #160

Open ghost opened 4 years ago

ghost commented 4 years ago

It'll say "pending" and won't even attempt to retry and then it'll return "Failed! (Network Error)"

settings: Latest Firefox, Violentmonkey, https://imgur.com/xRxW109

ccloli commented 4 years ago

Nope, in fact it's retrying but with no progress (it will keep the previous status until it downloads some content).

So when you see Pending... means it's now start downloading but doesn't receive anything yet, and yellow Failed (...) means it's now retrying to download again but with no progress too. Only red Failed (...) means it exceeds the maximum retry times.

label:question

------------------ Original ------------------ From: "pencilbook"notifications@github.com Date: Mon, Apr 27, 2020 03:23 AM To: "ccloli/E-Hentai-Downloader"E-Hentai-Downloader@noreply.github.com; Cc: "Subscribed"subscribed@noreply.github.com; Subject: [ccloli/E-Hentai-Downloader] Won't retry, spits out Failed! (NetworkError) (#160)

It'll say "pending" and won't even attempt to retry and then it'll return "Failed! (Network Error)"

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

ghost commented 4 years ago

Oh I see.

I'm pretty sure it used to say "retrying" before, was there a reason for it to change?

ccloli commented 4 years ago

Well, Pending... means it's downloading but haven't received data yet, and Downloading... means it receives some data now.

So just like the normal status, Failed status that shows as yellow means it stops old request and starts a new request but no data, and Retrying... means the request has got some data.

               get
               some
 +---------+   data  +-------------+  finish  +---------+
 | Pending +----+----> Downloading +----------> Succeed <-------+
 +---------+    |    +------+------+          +---------+       |
loading url &   |           |        get error                  |
start request   | get error +-----------------------+           |
                |           |                       |           |
                |       +---v----+            +-----+----+      |
                +-------> Failed +------------> Retrying +------+
                        +--------+     get    +----------+ finish
                     abort request &   some
                     start retrying    data
ghost commented 4 years ago

I am not sure if there's something wrong with my settings. Image 4, 14, 20, and 35 aren't loading at all for me. When I try to read those pages from the site, it won't load until I click on "Click here if the image fails loading" under the image. I always assumed when the script would retry, it would download the pages from the "Click here if the image fails loading" links but it's not doing that at all. What could be causing it?

Gallery: e-hentai.org/g/1620312/9cf711ab6f/

ghost commented 4 years ago

Suggestion: When the script fails to download an image after one attempt, it should download the image from "Click here if the image fails loading"

ccloli commented 4 years ago

Suggestion: When the script fails to download an image after one attempt, it should download the image from "Click here if the image fails loading"

Yes it will, once it hits the maximum retry time (default is 3) and you choose retry downloading. That means before the Failed goes red, it won't try that link as it may cost you more viewing limits (it hits original image server instead of H@H nodes). When other images are done, script will ask you to retry failed ones, that'll use the Click here if the image fails loading link.

ghost commented 4 years ago

I don't think that's happening anymore, a lot has changed on the site. I can retrieve the failed image by clicking on "Click here if the image fails loading", so there should be no reason for the script to fail. What's your FF version and are you using Greasemonkey or Violentmonkey?

ccloli commented 4 years ago

Maybe I didn't explain it well, so let's make it short.

When it download fails, it'll attempt to retry 3 times by default, at this time it won't reach Click here if the image fails loading link. If all of these retrys are failed, the script will ask you to try again, at this time it will reach Click here if the image fails loading link*.

*: unless you enabled Never get new image URL when failed to download image or Never send "nl" GET parameter when getting new image URL, which are disabled by default.