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
[extractor/rbgtum] fix m3u8 regex, add new hostname #7690
Conversation
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.
I can see a few issues in the IE unrelated to your PR. I'll refrain from suggesting changes for them unless you are interested in looking at them.
I am happy to take a look |
Co-authored-by: pukkandan <pukkandan.ytdlp@gmail.com>
Ok, it seems like there was a big change to the website layout yesterday, I have to adopt it now. how coincident :D |
The new ui should be working, a new json extractor added, with the old html parser as backup |
Co-authored-by: bashonly <88596187+bashonly@users.noreply.github.com>
This reverts commit 874fed6.
Co-authored-by: bashonly <88596187+bashonly@users.noreply.github.com>
Authored by: simon300000
IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information
This pr contains:
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 05a2890
Summary
🆕🛠️🧹
Improved and fixed
RbgTumIE
andRbgTumCourseIE
extractors to handle new domain and URL formats. Added tests and refactored code for clarity and robustness.Walkthrough
RbgTumIE
andRbgTumCourseIE
extractors to support the new domaintum.live
as an alternative tolive.rbg.tum.de
(link, link)_html_search_regex
call for them3u8
variable in theRbgTumIE
extractor (link)hostname
variable from the_match_valid_url
call instead of hardcoding thelive.rbg.tum.de
domain in theRbgTumCourseIE
extractor, to make it more flexible and consistent with theRbgTumIE
extractor (link, link)RbgTumIE
andRbgTumCourseIE
extractors to check the extraction of videos and courses from the new domaintum.live
(link, link)