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
[ie/bpb] Support more urls and extract more metadata #8119
Conversation
def _parse_vue_attributes(self, name, string, video_id): | ||
attributes = extract_attributes(self._search_regex(rf'(<{name}(?:"[^"]*?"|[^>])*>)', string, name)) | ||
|
||
for key, value in attributes.items(): | ||
if key.startswith(':'): | ||
attributes[key] = self._parse_json(value, video_id, transform_source=js_to_json, fatal=False) | ||
|
||
return attributes |
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.
Just a note that it might be useful to eventually generalize this and add it as an InfoExtractor
helper method, since it is very similar to ZaikoBaseIE._parse_vue_element_attrs
(but not similar enough as-is)
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
Authored by: Grub4K
IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information
This PR allows all possible url formats and extracts more metadata.
Fixes #8093
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?
Copilot Summary
🤖 Generated by Copilot at 73f6743
Summary
🎥🌐✨
This pull request improves the
BpbIE
extractor to support more media types and metadata from the Bundeszentrale für politische Bildung website. It also fixes a bug in thedetermine_ext
function by adding a missing entry for video/ogg in themimetype2ext
dictionary.Walkthrough
utils
module to map video/ogg to ogv extension (link)