node-downloader-helper icon indicating copy to clipboard operation
node-downloader-helper copied to clipboard

fix: make retry cover errors during headers and during body

Open mitom opened this issue 1 year ago • 3 comments

I believe this fixes one part of the issue in #115 and maybe helps #113 a bit as well, not sure if it entirely closes either but maybe one step closer.

The logic behind it is fairly simple - make the error handling during parsing the resposne be the same as the error handling of the http event listeners, the __onError function already takes care of setting the state to failed when we're out of retries and also emitting the error.

mitom avatar Jan 28 '24 13:01 mitom

Actually it looks like this is not ready to merge, the retry counter doesn't seem to be increasing so it goes infinitely.

mitom avatar Jan 29 '24 11:01 mitom

Actually it looks like this is not ready to merge, the retry counter doesn't seem to be increasing so it goes infinitely.

ok ok, i will check all these when i have some time including #115 and #113 , but unfortunately i am still unable to do a proper check and fix it, i will let you know as soon is possible

hgouveia avatar Jan 29 '24 13:01 hgouveia

Thank you, and no worries - I'm trying to help as much as I can, not trying to rush you!

mitom avatar Jan 29 '24 14:01 mitom