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

[GoPlay] Uses new API #6151

Merged
merged 1 commit into from
Feb 3, 2023
Merged

[GoPlay] Uses new API #6151

merged 1 commit into from
Feb 3, 2023

Conversation

jeroenj
Copy link
Contributor

@jeroenj jeroenj commented Feb 3, 2023

Uses the new API (api.goplay.be) for the GoPlay extractor. The old API (api.viervijfzes.be) seems to have been shut down. The API domain now matches the extractor name as well (see #3412 for the initial rename).

Fixes #6032

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 one of the following options:

  • 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?

@pukkandan
Copy link
Member

I assume you have run the tests?

@jeroenj
Copy link
Contributor Author

jeroenj commented Feb 3, 2023

Yes, and tested manually with credentials (since the tests are skipped).

@pukkandan pukkandan merged commit d27bde9 into yt-dlp:master Feb 3, 2023
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.

[GoPlay] Unable to download JSON metadata: HTTP Error 403: Forbidden
2 participants