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

[extractors/globalplayer] add Global Player extractors #6903

Merged
merged 7 commits into from Apr 26, 2023

Conversation

garret1317
Copy link
Collaborator

@garret1317 garret1317 commented Apr 24, 2023

IMPORTANT: PRs without the template will be CLOSED

Description of your pull request and other information

This PR adds support for Global Player, the streaming site for Global Media's radio stations and podcasts. (ironically, Global only operates in the UK*.)

It supports:

  • Live Radio
  • Radio Catch-up
  • Podcasts
  • "Live Playlists" - similar to live radio, but upfront about the fact that it's basically just someone's itunes playlist
  • Limited support for videos (not all metadata is extracted)

I've tried to use traverse_obj more than in my last PR, as that was one of the main changes made in the code review. I've also added a BaseIE for functions that multiple extractors use, but I may not have done it according to best practices etc - I don't know a lot of python, just enough to be dangerous™.

Some of the podcasts have ads. I don't think there's anything I can do about that.

Apparently, you can get higher quality audio with the mobile app. I haven't looked into this, and this PR doesn't implement it.

*the site is not region-locked.

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?

yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
Copy link
Member

@bashonly bashonly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can also relax the _VALID_URL regex to allow for queries, fragments, optional trailing slashes

yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
garret1317 and others added 2 commits April 26, 2023 15:45
Co-Authored-By: bashonly <88596187+bashonly@users.noreply.github.com>
also one flake8

Co-Authored-By: bashonly <88596187+bashonly@users.noreply.github.com>
Co-Authored-By: bashonly <88596187+bashonly@users.noreply.github.com>
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
yt_dlp/extractor/globalplayer.py Outdated Show resolved Hide resolved
Co-authored-by: pukkandan <pukkandan.ytdlp@gmail.com>
@bashonly bashonly merged commit 3064766 into yt-dlp:master Apr 26, 2023
11 checks passed
@Vangelis66 Vangelis66 mentioned this pull request Apr 27, 2023
5 tasks
dirkf added a commit to dirkf/youtube-dl that referenced this pull request May 3, 2023
dirkf added a commit to dirkf/youtube-dl that referenced this pull request May 3, 2023
@mikf mikf mentioned this pull request Jun 19, 2023
9 tasks
dirkf added a commit to dirkf/youtube-dl that referenced this pull request Jul 18, 2023
dirkf added a commit to dirkf/youtube-dl that referenced this pull request Jul 19, 2023
dirkf added a commit to dirkf/youtube-dl that referenced this pull request Jul 19, 2023
dirkf added a commit to ytdl-org/youtube-dl that referenced this pull request Jul 19, 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.

None yet

3 participants