mikf / gallery-dl

Command-line program to download image galleries and collections from several image hosting sites
GNU General Public License v2.0
11.64k stars 950 forks source link

Instagram video download error #405

Closed nmilo closed 5 years ago

nmilo commented 5 years ago

Running latest version 1.10.3 When you try to download instagram video it outputs following error

[urllib3.connectionpool][debug] https://www.instagram.com:443 "GET /p/[post_id]/ HTTP/1.1" 200 17253
[download][error] 'ytdl:' URLs are not supported/enabled
[instagram][error] An unexpected error occurred: AttributeError - 'NoneType' object has no attribute 'scheme'. Please run gallery-dl again with the --verbose flag, copy its output and report this issue on https://github.com/mikf/gallery-dl/issues .
[instagram][debug] 
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/gallery_dl/job.py", line 48, in run
    self.dispatch(msg)
  File "/usr/local/lib/python3.6/dist-packages/gallery_dl/job.py", line 93, in dispatch
    self.handle_url(url, kwds)
  File "/usr/local/lib/python3.6/dist-packages/gallery_dl/job.py", line 213, in handle_url
    if not self.download(url):
  File "/usr/local/lib/python3.6/dist-packages/gallery_dl/job.py", line 280, in download
    downloader = self.get_downloader(scheme)
  File "/usr/local/lib/python3.6/dist-packages/gallery_dl/job.py", line 300, in get_downloader
    if klass.scheme == "http":
AttributeError: 'NoneType' object has no attribute 'scheme'
iamleot commented 5 years ago

Hello nmilo,

nmilo writes:

Running latest version 1.10.3 When you try to download instagram video it outputs following error


[urllib3.connectionpool][debug] https://www.instagram.com:443 "GET /p/[post_id]/ HTTP/1.1" 200 17253
[download][error] 'ytdl:' URLs are not supported/enabled
[...]

This sounds like youtube-dl isn't available.

Can you please install it and retry? (I have tested with https://www.instagram.com/p/Bqxp0VSBgJg/ and it works here)

nmilo commented 5 years ago

Installed youtube-dl, it works now, thank you. Should it be in requirements.txt?

iamleot commented 5 years ago

It's only used for a couple extractors and an optional dependency so IMHO no (but it's documented in README).