Skip to content

Drop StreamSel headers, share yt-dlp identity argv - #47

Merged
Linuxuser67 merged 3 commits into
mainfrom
tidy-argv
Sep 18, 2026
Merged

Linuxuser67 merged 3 commits into
mainfrom
tidy-argv

Conversation

@Linuxuser67

Copy link
Copy Markdown
Owner

Cleanup with zero behavior delta (all pre-existing argv tests pass unmodified):

  • Drop StreamSel.headers: write-only since parts moved to the binary (verified zero readers). The HttpHeaders type stays for the live HLS path.
  • New ytdlp_identity_args helper (cookies + optional UA + end-of-options separator + page URL) replacing three hand-assembled copies in extraction, part and HLS spawns. Extraction keeps its exact behavior (no UA there — None); the live-resolve branch adopts it on rebase.

214 passing, clippy clean. Disjoint hunks from PR #46 (only shared context is the helper it should adopt).

@Linuxuser67
Linuxuser67 merged commit 1b0d55c into main Sep 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