-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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 for CableCastTV #9969
base: master
Are you sure you want to change the base?
Conversation
As noted in the issue, |
Can you give links to the others sites? |
as of now, I changed the regex to match any links that contains the domain cablecast.tv, I would like to add support again for https://wctv.wilmette.com/CablecastPublicSite/, but I need to know the general pattern for CableCastTV websites, if there a website that list all of them it would be helpful |
Ok, I change regex for the valid url to be more general, as of now it should match any url that contains 'cablecast.tv/show' or 'CablecastPublicSite/show/' inside the url string, based on what I found on google search this extractor should work with every website, but still I don't know the full list of website that may be using this back end, so some may be left out.. Websites that I tested and it worked: and added 2 of them as only url matching and one of them for a complete test, should I change that? |
IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information
Adds extractor for https://wctv.wilmette.com/CablecastPublicSite/
Fixes #9785
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?