dirkf
dirkf
We know what the patch is but there isn't a patched yt-dl release. If you install yt-dl using _pip_, the extractor modules will be installed somewhere on your disk. Then...
If you run the yt-dl instance installed with pip as in your original post, it should give you some output that will show where the files are installed. In that...
Did you also make the change from https://github.com/yt-dlp/yt-dlp/issues/5360#issuecomment-1292387041? The surrounding lines in the diff shown there may be slightly different but the pattern to be changed is the same.
The example page has a ld+json block but we don't find it because `utils.JSON_LD_RE` is defective: it doesn't allow for spaces around `=` in `attr=value` expressions in the HTML script...
In my test code I actually copied the `_search_json_ld()` method into the extractor and patched the regex into it. The PR will fix the original value, so as to avoid...
1. Existing test 1 has a clip ID as the value of the `videoUrl` query parameter. 2. Existing test 2 has a video iframe whose `src` link contains a clip...
Is the PR affected: https://github.com/yt-dlp/yt-dlp/issues/3325#issuecomment-1254073311 ?
Great! >... I can split the PR if you'd like ... A mass update is fine when an extractor hasn't really been working for ages. Different case for the core...
Test case: https://github.com/ytdl-org/youtube-dl/issues/31276#issuecomment-1265311716 The extractor gets some 403s. If these are expected (eg out of region, not logged in), we could try to catch the warnings. The `expected_status` parameter isn't...
Also you might consider #31085.