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

[extractor/europarl] Rewrite extractor to use new api url #7114

Merged
merged 3 commits into from
May 29, 2023

Conversation

HobbyistDev
Copy link
Contributor

@HobbyistDev HobbyistDev commented May 24, 2023

IMPORTANT: PRs without the template will be CLOSED

Description of your pull request and other information

This PR change the api url from vuplay to arbor.nl. This PR rewrite almost all component of the extractor. This PR also get different id from the old api and afaik, there is no way to know how the old id can get or linked. Embed url is not supported anymore because the embed url need js to get processed.

Fixes #6396

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:

  • I am the original author of this code and I am willing to release it under Unlicense
  • I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

Copilot Summary

🤖 Generated by Copilot at 759a787

Summary

🔄🔧📺

Update EuropaTVIE extractor to use new API and JSON format, simplify URL pattern, and add live stream support. This improves compatibility and functionality for the europa.eu website.

EuropaTVIE changed
New API and JSON
Live streams in autumn

Walkthrough

  • Simplify _VALID_URL pattern and update test cases (link, link)
  • Import traverse_obj function from utils module (link)
  • Rewrite _real_extract method to use new API and JSON structure (link)
  • Update or add test cases for new API and JSON structure (link)

@pukkandan pukkandan force-pushed the master branch 2 times, most recently from ee280c7 to 7aeda6c Compare May 24, 2023 18:09
@pukkandan pukkandan merged commit 0378997 into yt-dlp:master May 29, 2023
aalsuwaidi pushed a commit to aalsuwaidi/yt-dlp that referenced this pull request Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

extractor/europarl no longer works - new video provider (arbor.nl)
2 participants