Skip to content

fix: tighten 403 retry and limit progressive skip to YouTube with DASH builder - #197

Merged
firsttris merged 1 commit into
masterfrom
ccr-1332d384-9ub9jp
Oct 20, 2026
Merged

firsttris merged 1 commit into
masterfrom
ccr-1332d384-9ub9jp

Conversation

@firsttris

@firsttris firsttris commented Oct 20, 2026 •

Copy link
Copy Markdown
Owner

Follow-up to #196 by @zibaone:

  • 403 retry: _fetch_range retries only on HTTP 403, waiting 3 s and then 6 s (the 9 s delay was never reached). The unreachable fallback paths are removed.
  • Skipping itag 18/22: only for YouTube results, and only when the DASH builder can actually play the video (DASH builder enabled and ISA supports mpd). Users without the DASH builder or inputstream.adaptive keep the muxed fallback, and other extractors that use numeric format ids are not affected.
  • Tests for a persistent 403 and for the narrower 18/22 skip.

pytest: 349 passed, 1 skipped.

🤖 Generated with Claude Code

https://claude.ai/code/session_0182LKQrJ9qZVuZqoLxppVdG

@codecov-commenter

codecov-commenter commented Oct 20, 2026 •

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.65%. Comparing base (480c7db) to head (70f51c4).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #197      +/-   ##
==========================================
+ Coverage   84.45%   84.65%   +0.20%     
==========================================
  Files          14       14              
  Lines        2612     2601      -11     
==========================================
- Hits         2206     2202       -4     
+ Misses        406      399       -7     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…H builder

- _fetch_range: retry only on HTTP 403 with 3 s / 6 s backoff (the 9 s
  delay was never reached), drop the unreachable fallback paths.
- Skip itag 18/22 only for YouTube results and only when the DASH builder
  can actually play the video (dashbuilder enabled and ISA supports mpd);
  users without the DASH builder or ISA keep the muxed fallback, and other
  extractors with numeric format ids are unaffected.
- Tests for a persistent 403 and for the gated progressive skip.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0182LKQrJ9qZVuZqoLxppVdG
@firsttris
firsttris force-pushed the ccr-1332d384-9ub9jp branch from 5e5789c to 70f51c4 Compare October 20, 2026 04:45
@firsttris firsttris changed the title fix: YouTube 403 — retry on soft block + skip removed progressive formats (#196) fix: tighten 403 retry and limit progressive skip to YouTube with DASH builder Oct 20, 2026
@firsttris
firsttris merged commit 87115e3 into master Oct 20, 2026
2 checks passed
@firsttris
firsttris deleted the ccr-1332d384-9ub9jp branch October 20, 2026 04:46
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.

3 participants