-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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.
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
Add YouTube client for 1080p muxed videos #9546
Conversation
I am opposed to adding this to the default list. Opinions? @bashonly @coletdjnz Also, make sure to document in readme |
This comment was marked as outdated.
This comment was marked as outdated.
|
I'm not too concerned about it being on by default or not, i just added it there while testing with vrchat to check the returned streams were compatible with avpro easily, as long as it can be enabled with a command line option I'm fine with it not being on by default. I'm also not too sure what the difference between base_clients vs defaults is as I'm unfamiliar with the codebase. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as resolved.
This comment was marked as resolved.
"Wrong player response" issue has been confirmed to not be due to this PR and should be tracked separately |
You can also get the 1080p muxed videos by using a safari user agent with the web client, so could also add a duplicate of web client with the safari user agent.
|
Co-authored-by: coletdjnz <coletdjnz@protonmail.com>
Co-authored-by: bashonly <88596187+bashonly@users.noreply.github.com>
Interesting, is it the same streams as this returns? |
web client only, using safari UA:
media connect frontend only:
(no need to add safari web client version atm though, can use --user-agent for that) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
try |
Yep, works with that, thanks |
Is there a way to only impersonate safari for a specific extractor? |
there will be when #9474 is merged |
- Possibly allows 1080p non-adaptive formats to be used - yt-dlp/yt-dlp#9546
yt_dlp/extractor/youtube.py
Outdated
@@ -238,6 +238,16 @@ | |||
}, | |||
'INNERTUBE_CONTEXT_CLIENT_NAME': 85 | |||
}, | |||
# This client has 1080p non split stream videos | |||
'media_connect_frontend': { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as pukkandan mentioned on discord, _
has special meaning in client names, so we need to change this to something like mediaconnect
Description of your pull request and other information
Adds the ability for yt-dlp to return premuxed 1080p results for videos when available.
Template
Before submitting a pull request make sure you have:
In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:
What is the purpose of your pull request?