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

New Nicovideo Issue #14135

Closed
3 of 8 tasks
dkl3 opened this issue Sep 6, 2017 · 2 comments · Fixed by #14497
Closed
3 of 8 tasks

New Nicovideo Issue #14135

dkl3 opened this issue Sep 6, 2017 · 2 comments · Fixed by #14497

Comments

@dkl3
Copy link

dkl3 commented Sep 6, 2017

Please follow the guide below

  • You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
  • Put an x into all the boxes [ ] relevant to your issue (like this: [x])
  • Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.09.02. 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.09.02

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

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:

youtube-dl http://www.nicovideo.jp/watch/sm18238488 --username USER --password USER -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['http://www.nicovideo.jp/watch/sm18238488', '--username', 'PRIVATE', '--password', 'PRIVATE', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.09.02
[debug] Python version 3.5.2 - Linux-4.4.0-81-generic-i686-with-LinuxMint-18.1-serena
[debug] exe versions: ffmpeg 2.8.11-0ubuntu0.16.04.1, ffprobe 2.8.11-0ubuntu0.16.04.1
[debug] Proxy map: {}
[niconico] Logging in
[niconico] sm18238488: Downloading webpage
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl", line 11, in
sys.exit(main())
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/init.py", line 465, in main
_real_main(argv)
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/init.py", line 455, in _real_main
retcode = ydl.download(all_urls)
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/YoutubeDL.py", line 1958, in download
url, force_generic_extractor=self.params.get('force_generic_extractor', False))
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/YoutubeDL.py", line 776, in extract_info
ie_result = ie.extract(url)
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/common.py", line 434, in extract
ie_result = self._real_extract(url)
File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/niconico.py", line 399, in _real_extract
uploader_id = get_video_info(['ch_id', 'user_id']) or owner.get('id')
AttributeError: 'NoneType' object has no attribute 'get'


Description of your issue, suggested solution and other information

I try and download a video with NicoVideo, and it gives this error. I tried to check for a recent bug report on this but i didn't see anything. Does anyone know the cause? thank you.

@ghost
Copy link

ghost commented Sep 7, 2017

also had this bug

the bug is this part always assuming a video has an owner (i think?)

i worked around it in my local copy by doing this:

owner = api_data.get('owner', {})
uploader_id = get_video_info(['ch_id', 'user_id']) or owner and owner.get('id')
uploader = get_video_info(['ch_name', 'user_nickname']) or owner and owner.get('nickname')

@dkl3
Copy link
Author

dkl3 commented Sep 8, 2017

Maybe that can be merged with this youtube-dl?

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 a pull request may close this issue.

1 participant