Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

zstd content-encoding #9525

Open
6 of 9 tasks
selfisekai opened this issue Mar 24, 2024 · 3 comments
Open
6 of 9 tasks

zstd content-encoding #9525

selfisekai opened this issue Mar 24, 2024 · 3 comments
Labels
enhancement New feature or request networking core networking related

Comments

@selfisekai
Copy link
Contributor

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

  • I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

Provide a description that is worded well enough to be understood

chromium M123 (latest stable) introduces zstd content-encoding. this provides opportunity for faster http transfers, as support within websites will be rolling out

Provide verbose output that clearly demonstrates the problem

  • Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
  • If using API, add 'verbose': True to YoutubeDL params instead
  • Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output

No response
@selfisekai selfisekai added enhancement New feature or request triage Untriaged issue labels Mar 24, 2024
@Grub4K
Copy link
Member

Grub4K commented Mar 24, 2024

Note that urllib3 has support for zstd using urllib3[zstd]

@Grub4K Grub4K added networking core networking related and removed triage Untriaged issue labels Mar 24, 2024
@pukkandan
Copy link
Member

pukkandan commented Mar 25, 2024 •
edited

We can also add it for urllib with zstandard (which is what urllib3 also uses). See brotli impl for reference.

cc @coletdjnz

@coletdjnz
Copy link
Member

We can enable it for requests. Fingerprinting concerns don't really matter now that we have impersonation. Urllib handler is soft deprecated and in maintainence-only state more or less. I wouldn't bother updating it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request networking core networking related
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@pukkandan @selfisekai @coletdjnz @Grub4K