Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove the default download timeout of 5 minutes for a single file #81

Merged
merged 1 commit into from
Jan 5, 2022

Conversation

Cadair
Copy link
Owner

@Cadair Cadair commented Dec 15, 2021

Also close the request which gets the headers before opening new ones when transferring data.

@codecov
Copy link

codecov bot commented Dec 15, 2021

Codecov Report

Merging #81 (7188fa8) into main (95db21b) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #81   +/-   ##
=======================================
  Coverage   89.64%   89.64%           
=======================================
  Files           4        4           
  Lines         425      425           
=======================================
  Hits          381      381           
  Misses         44       44           
Impacted Files Coverage Δ
parfive/downloader.py 91.91% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 95db21b...7188fa8. Read the comment docs.

Also close the request which gets the headers before opening new ones when
transferring data.
@dstansby
Copy link
Contributor

Maybe closing the original request will solve #65 ??

@Cadair
Copy link
Owner Author

Cadair commented Jan 5, 2022

We can but hope that it helps 65. It should close the connection eventually i.e. when the whole file has transferred anyway.

@Cadair Cadair merged commit 1f2cc6a into main Jan 5, 2022
@Cadair Cadair deleted the timeouts branch January 5, 2022 10:26
@Cadair Cadair changed the title Remove the default total download timeout Remove the default download timeout of 5 minutes for a single file Jan 5, 2022
@Cadair Cadair added the bug A report of or a fix for a bug or unwanted behaviour label Jan 5, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug A report of or a fix for a bug or unwanted behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants