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

[ie/qqmusic] Fix QQMusic extractors for new URLs #9768

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

c-basalt
Copy link
Contributor

@c-basalt c-basalt commented Apr 23, 2024

IMPORTANT: PRs without the template will be CLOSED

Description of your pull request and other information

Overhauled extractors to support new urls of the site.

  • Replaced broken old API
  • Changed RegEx to match current URL patterns

(side note: API endpoint musicu.fcg works the same as musics.fcg but does not requires a signature, so I removed signing algorithm from earlier commit of this patch to https://gist.github.com/c-basalt/9bc9feeed8e39e928b6607ddf01840b5)

Fixes #9336

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?

@seproDev seproDev added the site-bug Issue with a specific website label Apr 23, 2024
@c-basalt c-basalt marked this pull request as ready for review April 23, 2024 21:04
@c-basalt c-basalt changed the title [extractor/qqmusic] Fix QQMusic extractors for new URLs [ie/qqmusic] Fix QQMusic extractors for new URLs Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
site-bug Issue with a specific website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The latest qq music web link is not recognized
2 participants