Skip to content

Report incomplete downloads and preserve interrupted transfers - #181

Merged
tompollard merged 3 commits into
mainfrom
fix/download-failure-reporting
Oct 18, 2026
Merged

tompollard merged 3 commits into
mainfrom
fix/download-failure-reporting

Conversation

@tompollard

Copy link
Copy Markdown
Member

Summary

  • Make Python downloads raise DownloadError when any selected file fails, with paths, causes, and counts.
  • Return CLI status 1 for incomplete downloads and status 130 for Ctrl-C.
  • Keep incomplete HTTP and S3 files after interruption; document that explicit AWS downloads currently restart on the next attempt.
  • Update the README and add Python and CLI regression tests.

Tests

  • env/bin/python -m pytest -q — 135 passed, 1 skipped
  • git diff --check — passed

Scope

This is the first, bounded downloader correctness PR. Range handling, manifest path validation, credential scoping, and storage selection are follow-up work.

@tompollard
tompollard merged commit 6f7cb11 into main Oct 18, 2026
2 checks passed
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.

1 participant