AlexAplin / nndownload

Download and process links from Niconico (nicovideo.jp)
MIT License
223 stars 29 forks source link

Downloading a single video works fine but download a user playlist yields an error (running latest version). #101

Closed psydere closed 3 years ago

psydere commented 3 years ago

So I wanted to try the new --playlist-start option ( https://github.com/AlexAplin/nndownload/issues/98 ) but it seems like I've ran into some bugs that are totally unrelated to that.

I'm running the version that has https://github.com/AlexAplin/nndownload/commit/05af001e65204412d311eef75e5ef5454c8412d4 as the latest commit.

Take the sample video shown in the Readme:

python nndownload.py -o{uploader_id}" - "{uploader}\{published}{title}{id}.{ext} -f -u USERNAME -p PASSWORD https://www.nicovideo.jp/watch/sm35249846

works fine even with all the attribute (I did not use --playlist-start since it's just a single video).

But when I try to download from the uploader of said video,

python nndownload.py -o{uploader_id}" - "{uploader}\{published}{title}{id}.{ext} -f -u USERNAME -p PASSWORD https://www.nicovideo.jp/user/4762111

  File "B:\nndownload-master\nndownload\nndownload.py", line 1699, in main
    process_url_mo(session, url_mo)
  File "B:\nndownload-master\nndownload\nndownload.py", line 1622, in process_url_mo
    request_user(session, url_id)
  File "B:\nndownload-master\nndownload\nndownload.py", line 957, in request_user
    video_api_request.raise_on_status()
AttributeError: 'Response' object has no attribute 'raise_on_status'

I even tried without any fancy options python nndownload.py https://www.nicovideo.jp/user/4762111

Got the same erroneous output as above (it only appeared after I've log in successfully).