Skip to content

Use explicit default YouTube player clients - #454

Merged
Linuxuser67 merged 2 commits into
mainfrom
fix/ytdlp-explicit-default-player-client
Oct 18, 2026
Merged

Linuxuser67 merged 2 commits into
mainfrom
fix/ytdlp-explicit-default-player-client

Conversation

@Linuxuser67

@Linuxuser67 Linuxuser67 commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Set yt-dlp's YouTube player-client selector to default,-web instead of only -web.
  • Keep the default client rotation while explicitly excluding web, whose SABR-only formats can fail extraction in affected yt-dlp versions.
  • Update the argv expectations in src/video_tests.rs to match the explicit selector.

Validation

  • Verified the branch is one commit ahead of the current main, touching only src/video_tools.rs and src/video_tests.rs.
  • Updated the four stale test expectations responsible for the three reported failures.
  • The test suite has not been run locally here; check the CI result before merging.

@Linuxuser67
Linuxuser67 force-pushed the fix/ytdlp-explicit-default-player-client branch from 5e677f4 to 1d237fa Compare October 18, 2026 08:27
@Linuxuser67
Linuxuser67 merged commit 84f4bb7 into main Oct 18, 2026
5 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