Skip to content

[pull] dev from TeamNewPipe:dev#83

Merged
pull[bot] merged 5 commits intoabsurdlylongusername:devfrom
TeamNewPipe:dev
Jan 9, 2026
Merged

[pull] dev from TeamNewPipe:dev#83
pull[bot] merged 5 commits intoabsurdlylongusername:devfrom
TeamNewPipe:dev

Conversation

@pull
Copy link

@pull pull bot commented Jan 9, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

dustdfg and others added 5 commits January 2, 2026 12:25
When resuming a download after interruption, if the server returns
HTTP 200 (full resource) instead of HTTP 206 (partial content), the
code correctly resets mMission.done but fails to reset the 'start'
variable. This causes the subsequent file seek to use a stale offset,
writing new data at incorrect positions.

This bug causes file corruption for large downloads (>5GB) that are
interrupted and resumed, particularly when:
- Switching between WiFi networks
- Server CDN returning different responses
- Connection drops during long downloads

The corruption manifests as duplicate data regions in the file,
which for MP4 downloads results in multiple MOOV atoms and
broken seek functionality.

Fix: Reset start=0 when HTTP 200 is received, ensuring the file
write position correctly restarts from the beginning of the current
resource.
The workflow can be triggered by creating a comment on a merged PR: /backport <TARGET_BRANCH>
The backport can only be triggered by people with write access to the repository.

Co-authored-by: AbsurdlyLongUsername <22662897+absurdlylongusername@users.noreply.github.com>
Use "factory" method for creating db migrations
Fix download resume corruption when server returns HTTP 200
@pull pull bot locked and limited conversation to collaborators Jan 9, 2026
@pull pull bot added the ⤵️ pull label Jan 9, 2026
@pull pull bot merged commit 6941248 into absurdlylongusername:dev Jan 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants