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

[Youku] Crash when episode count is wrong #15135

Closed
CoXier opened this issue Jan 1, 2018 · 4 comments
Closed

[Youku] Crash when episode count is wrong #15135

CoXier opened this issue Jan 1, 2018 · 4 comments

Comments

@CoXier
Copy link
Contributor

@CoXier CoXier commented Jan 1, 2018

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.12.31. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

  • I've verified and I assure that I'm running youtube-dl 2017.12.31

Before submitting an issue make sure you have:

  • At least skimmed through the README, most notably the FAQ and BUGS sections
  • Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

  • Bug report (encountered problems with youtube-dl)
  • Site support request (request for adding support for a new site)
  • Feature request (request for a new functionality)
  • Question
  • Other

Here is a test url: http://list.youku.com/show/id_z20eb4acaf5c211e3b2ad.html

youtube-dl http://list.youku.com/show/id_z20eb4acaf5c211e3b2ad.html\?spm\=a2h0j.8191423.module_basic_title.5\~A\!2 -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'http://list.youku.com/show/id_z20eb4acaf5c211e3b2ad.html?spm=a2h0j.8191423.module_basic_title.5~A!2', u'-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.12.23
[debug] Python version 2.7.13 - Darwin-17.2.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.3.2, ffprobe 3.3.2
[debug] Proxy map: {}
[youku:show] z20eb4acaf5c211e3b2ad: Downloading webpage
[youku:show] z20eb4acaf5c211e3b2ad: Downloading initial playlist data page
[youku:show] z20eb4acaf5c211e3b2ad: Downloading playlist data page 2
ERROR: An extractor error has occurred. (caused by KeyError(u'html',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 438, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youku.py", line 292, in _real_extract
    'stage': reload_id,
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youku.py", line 254, in _extract_entries
    transform_source=lambda s: js_to_json(strip_jsonp(s)))['html']
KeyError: u'html'
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 784, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 451, in extract
    raise ExtractorError('An extractor error has occurred.', cause=e)
ExtractorError: An extractor error has occurred. (caused by KeyError(u'html',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

The episode count is 16 however it has three modules 1-40 41-80 81-86.

Obviously it's youku's fault.

@dstftw
Copy link
Collaborator

@dstftw dstftw commented Jan 1, 2018

Carefully read new issue template and provide all requested information.

@dstftw dstftw closed this Jan 1, 2018
@dstftw dstftw added the incomplete label Jan 1, 2018
@CoXier
Copy link
Contributor Author

@CoXier CoXier commented Jan 1, 2018

I have updated information. @dstftw

@dstftw dstftw reopened this Jan 1, 2018
@dstftw dstftw removed the incomplete label Jan 1, 2018
@CoXier
Copy link
Contributor Author

@CoXier CoXier commented Jan 1, 2018

CoXier/youtube-dl@5936bc6 solves this issue.

@CoXier CoXier closed this Jan 1, 2018
@dstftw
Copy link
Collaborator

@dstftw dstftw commented Jan 1, 2018

Do not close the issue when fix is not merged.

@dstftw dstftw reopened this Jan 1, 2018
@ytdl-org ytdl-org locked and limited conversation to collaborators Jan 1, 2018
@ytdl-org ytdl-org unlocked this conversation Jan 4, 2018
@yan12125 yan12125 closed this in da35331 Jan 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.