Skip to content

Release: Fix videos re-downloading after SponsorBlock timeouts or empty subtitles, plus a security update - #974

Merged
dialmaster merged 8 commits into
mainfrom
dev
Oct 27, 2026
Merged

dialmaster merged 8 commits into
mainfrom
dev

Conversation

@dialmaster

@dialmaster dialmaster commented Oct 27, 2026 •

Copy link
Copy Markdown
Collaborator

This release fixes two cases where Youtarr marked a video as failed even though it downloaded fine, then downloaded it again on the next run: when a SponsorBlock lookup failed, and when YouTube sent back an empty subtitle file. It also updates a dependency that let a single request crash the server.

fixes

dialmaster and others added 8 commits October 22, 2026 20:00
fix(deps): patch body-parser and multer security issues
Recognize WARNING lines before the final SponsorBlock ERROR, reproduce all
three extractor retries in the executor tests, and cover an unrelated
video failure that must not inherit a false HTTP 403 auto-retry.

Clarify that the SponsorBlock count appears in the live completion message.
fix: keep downloaded videos when SponsorBlock is unavailable
When YouTube sent back an empty subtitle file, Youtarr marked the
video as failed even though it downloaded fine, then downloaded it
again on every run.

Now the video is kept and the job shows a subtitle warning instead.
If the video file itself is missing, it still counts as failed and is
tried again next run.

Refs: #958
…ks-video-failed

fix: don't fail videos over an empty subtitle
@dialmaster dialmaster changed the title Release Release: Fix videos re-downloading after SponsorBlock timeouts or empty subtitles, plus a security update Oct 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📊 Test Coverage Report

Backend Coverage

Type Coverage Status
Lines 91.28% 🟢
Statements 91.17% 🟢
Functions 91.13% 🟢
Branches 85.19% 🟢

Frontend Coverage

Type Coverage Status
Lines 89.77% 🟢
Statements 88.23% 🟢
Functions 81.99% 🟢
Branches 82.92% 🟢

Coverage Requirements

  • Minimum threshold: 70% line coverage
  • Backend: ✅ Passes
  • Frontend: ✅ Passes

Coverage report generated for commit 14da9dc

@dialmaster
dialmaster merged commit c7ba55c into main Oct 27, 2026
13 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.

2 participants