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

SteamIE videos doesn't set the extractor key in the video info_dict #877

Closed
jaimeMF opened this issue Jun 7, 2013 · 1 comment
Closed

SteamIE videos doesn't set the extractor key in the video info_dict #877

jaimeMF opened this issue Jun 7, 2013 · 1 comment
Labels
bug

Comments

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Jun 7, 2013

$ ./y-dl "http://store.steampowered.com/app/204450/" --write-thumbnail -v
[debug] System config: []
[debug] User config: ['--keep-video']
[debug] Command-line args: ['http://store.steampowered.com/app/204450/', '--write-thumbnail', '-v']
[debug] youtube-dl version 2013.05.23
[debug] Git HEAD: a23d635
[debug] Python version 2.7.5 - Darwin-11.4.2-x86_64-i386-64bit
[debug] Proxy map: {}
[Steam] Confirming age
[Steam] 204450: Downloading webpage
[download] Downloading playlist: Call of Juarez® Gunslinger
[Steam] playlist 'Call of Juarez® Gunslinger': Collected 1 video ids (downloading 1 of them)
[download] Downloading video #1 of 1
Traceback (most recent call last):
  File "./y-dl", line 6, in <module>
    youtube_dl.main()
  File "/Users/jaimemf93/Developer/proyectos/youtube-dl/youtube-dl/youtube_dl/__init__.py", line 603, in main
    _real_main(argv)
  File "/Users/jaimemf93/Developer/proyectos/youtube-dl/youtube-dl/youtube_dl/__init__.py", line 587, in _real_main
    retcode = fd.download(all_urls)
  File "/Users/jaimemf93/Developer/proyectos/youtube-dl/youtube-dl/youtube_dl/FileDownloader.py", line 719, in download
    videos = self.extract_info(url)
  File "/Users/jaimemf93/Developer/proyectos/youtube-dl/youtube-dl/youtube_dl/FileDownloader.py", line 477, in extract_info
    return self.process_ie_result(ie_result, download=download)
  File "/Users/jaimemf93/Developer/proyectos/youtube-dl/youtube-dl/youtube_dl/FileDownloader.py", line 554, in process_ie_result
    for r in ie_result['entries']
  File "/Users/jaimemf93/Developer/proyectos/youtube-dl/youtube-dl/youtube_dl/FileDownloader.py", line 544, in process_ie_result
    extra_info=extra)
  File "/Users/jaimemf93/Developer/proyectos/youtube-dl/youtube-dl/youtube_dl/FileDownloader.py", line 506, in process_ie_result
    self.process_info(ie_result)
  File "/Users/jaimemf93/Developer/proyectos/youtube-dl/youtube-dl/youtube_dl/FileDownloader.py", line 682, in process_info
    (info_dict['extractor'], info_dict['id']))
KeyError: 'extractor'

Currently we only set the extractor for the top dictionary we get from ie.extract_info. If we get a playlist with the videos info already extracted, we must make sure they have the extractor key set.

@jaimeMF
Copy link
Collaborator Author

@jaimeMF jaimeMF commented Jun 8, 2013

Closed in 8027175

@jaimeMF jaimeMF closed this Jun 8, 2013
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
1 participant
You can’t perform that action at this time.