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

[GBNews]Add new extractor for GB News TV channel #29432

Merged
merged 3 commits into from Feb 22, 2024
Merged

Conversation

dirkf
Copy link
Contributor

@dirkf dirkf commented Jun 29, 2021

Please follow the guide below

  • You will be asked some questions, please read them carefully and answer honestly
  • Put an x into all the boxes [ ] relevant to your pull request (like that [x])
  • Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

In order to be accepted and merged into youtube-dl 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?

  • Bug fix
  • Improvement
  • New extractor
  • New feature

Description of your pull request and other information

Issue #29341 identified the need for an extractor for the new GB News channel from the UK, and demonstrated extraction using a shell script.

This PR adds an extractor for the two types of GB News URLs (clips, live) shown in the issue, using the procedure from the issue.

The live stream appears to operate OK with a shell command line piping yt-dl output to mpv, but hasn't been extensively exercised.

Closes #29341.

@ghost
Copy link

ghost commented Jul 2, 2021

Small issue in the URL regex, videos on gbnews are not exclusive to /shows, and can also appear as other types, as can be seen at https://www.gbnews.uk/news/sedgefield-company-creates-revolutionary-covid-detection-machine/109879
Other than that, seems good 👍

@dirkf
Copy link
Contributor Author

dirkf commented Jul 10, 2021

Apparently any text (let's say alphanumeric and _) can go in place of shows and the link redirects to the preferred version.

Allow alphanumeric and _ in place of `shows`, which redirect to site's preferred URL
Copy link
Contributor Author

@dirkf dirkf left a comment

Choose a reason for hiding this comment

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

Update for 2024

youtube_dl/extractor/extractors.py Outdated Show resolved Hide resolved
youtube_dl/extractor/gbnews.py Outdated Show resolved Hide resolved
youtube_dl/extractor/gbnews.py Outdated Show resolved Hide resolved
youtube_dl/extractor/gbnews.py Outdated Show resolved Hide resolved
youtube_dl/extractor/gbnews.py Outdated Show resolved Hide resolved
youtube_dl/extractor/gbnews.py Outdated Show resolved Hide resolved
youtube_dl/extractor/gbnews.py Outdated Show resolved Hide resolved
youtube_dl/extractor/gbnews.py Outdated Show resolved Hide resolved
youtube_dl/extractor/gbnews.py Outdated Show resolved Hide resolved
youtube_dl/extractor/gbnews.py Outdated Show resolved Hide resolved
@dirkf dirkf mentioned this pull request Feb 22, 2024
11 tasks
@dirkf dirkf merged commit 70f230f into ytdl-org:master Feb 22, 2024
14 checks passed
github-actions bot added a commit to hellopony/youtube-dl that referenced this pull request Feb 22, 2024
* https://github.com/ytdl-org/youtube-dl:
  [caffeine.tv] Add new extractor (ytdl-org#32514)
  [GBNews]Add new extractor for GB News TV channel (ytdl-org#29432)
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.

Add support for gbnews.uk
1 participant