Skip to content
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

Fix support for redbull.com (redbull.tv has moved to redbull.com) #22037

Closed
5 tasks done
gschaffner opened this issue Aug 8, 2019 · 5 comments
Closed
5 tasks done

Fix support for redbull.com (redbull.tv has moved to redbull.com) #22037

gschaffner opened this issue Aug 8, 2019 · 5 comments
Labels
site-support-request Add extractor(s) for a new domain

Comments

@gschaffner
Copy link

Checklist

  • I'm reporting a new site support request
  • I've verified that I'm running youtube-dl version 2019.08.02
  • I've checked that all provided URLs are alive and playable in a browser
  • I've checked that none of provided URLs violate any copyrights
  • I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Description

RedBullTV is supported by youtube-dl. However, it appears that links to videos on redbull.tv redirect to redbull.com rather than redbull.tv. Trying to download these videos results in an Unsupported URL error.

It also appears that some redbull.tv links (such as https://www.redbull.tv/video/AP-1Q6XCDTAN1W11 and https://www.redbull.tv/video/AP-1PMHKJFCW1W11 from the RedBullTV extractor's tests) now redirect to redbull.com with a different URL format.

@gschaffner gschaffner added the site-support-request Add extractor(s) for a new domain label Aug 8, 2019
@gschaffner
Copy link
Author

It appears that multiple of the tests in redbulltv.py are no longer playable in browser.

@gschaffner
Copy link
Author

Additionally, redbull.tv is now redirecting to https://www.redbull.com/int-en/discover (for my region, at least). Thus, I believe it is safe to remove support for redbull.tv URLs in favor of redbull.com ones.

@gschaffner gschaffner changed the title Redbull.com support Fix support for redbull.com (redbull.tv has moved to redbull.com) Aug 9, 2019
@Boris-de
Copy link
Contributor

Boris-de commented Oct 3, 2019

Thanks! The referenced PR works for me with one exception. Sometimes the URLs do not have an "AP-..." id, but instead have a slug in that place. Can't find the in the HTML, so not sure how to get it.
Example: https://www.redbull.com/int-en/episodes/age-of-ondra-part-3

@Boris-de
Copy link
Contributor

Boris-de commented Oct 3, 2019

One more thing: #21781 has URLs that are also not supported yet, but as they have an AP-id they look like they could be added by changing the regex.

@ghost ghost mentioned this issue Dec 2, 2019
5 tasks
@gschaffner
Copy link
Author

@Boris-de Thanks for the feedback! I've updated #22063 and slugs like that should work now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
site-support-request Add extractor(s) for a new domain
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants