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

fix warning (channel id) #21003

Merged
merged 2 commits into from May 8, 2019
Merged

fix warning (channel id) #21003

merged 2 commits into from May 8, 2019

Conversation

pypy-vrc
Copy link

@pypy-vrc pypy-vrc commented May 8, 2019

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

Fixed to get channel_id from video_details.

This fixes #20982

@Videonauth
Copy link

Can confirm, this fixes the issue with channel ID.

@dstftw dstftw merged commit b45a9e6 into ytdl-org:master May 8, 2019
@pypy-vrc pypy-vrc deleted the issue20982 branch May 9, 2019 05:10
@ivan
Copy link
Contributor

ivan commented May 10, 2019

On Vevo videos like MfQoejFPgc0 and 4ohvXEkMiOk, video_details.get('channelId') and the channelId meta tag point to the correct channel with a non-empty upload playlist (UCsbkOzMoqB7LwJClG6d28Lg), while the subscribe button's data-channel-external-id= points to some kind of Vevo meta channel (UCDcyb4c29bZcQkldeQyY_YQ) with an empty upload playlist. I believe this makes the data-channel-external-id= value not particularly reliable for getting the correct value.

@LegeNet
Copy link

LegeNet commented May 10, 2019

OK. Also experienced this. Nice that there now is a fix. Experienced with: youtube-dl --verbose --write-sub --sub-lang en,sv --format 18 "http://www.youtube.com/watch?v=kq1PbgeBoQ4"

Is from 2008 and has 2,806,445 views.

This is not a serious issue as the film does download OK, but gives the message WARNING: unable to extract channel id; please report this issue on https://yt-dl.org/bug

fireattack added a commit to fireattack/Particle that referenced this pull request May 24, 2019
YouTube recently removed "channelID" meta tag for video page from personal channels (hence the previous commit). This doesn't seem a big concern as you can get the ID from username's hyperlink just fine.

However, today I learned that VEVO channels still have it - and it is different form the ID you can get from username's hyperlink:

actually, username for VEVO videos now points to a dummy artist channel page (that doesn't show uploads).

This change will make it to check channelID meta tag first. so it can correctly link to and get actual uploaded video number from the actual VEVO channel, instead of the dummy artist page. If none found (like the case for personal channel), it falls back to hyperlink on uploader's name.

See comments in ytdl-org/youtube-dl#21003
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.

WARNING: unable to extract channel id
5 participants