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

FunnyOrDie extractor not finding thumbnails #2856

Closed
jillh510 opened this issue May 5, 2014 · 1 comment
Closed

FunnyOrDie extractor not finding thumbnails #2856

jillh510 opened this issue May 5, 2014 · 1 comment

Comments

@jillh510
Copy link

@jillh510 jillh510 commented May 5, 2014

python -m youtube_dl -v --skip-download --write-info-json --no-playlist -f mp4 http://www.funnyordie.com/videos/e402820827/please-use-this-song-jon-lajoie extracts the video properly but not a thumbnail. Here's the resulting JSON:
{"display_id": "e402820827", "extractor": "FunnyOrDie", "description": "Please use this to sell something", "format": "0 - unknown", "format_id": "0", "playlist_index": null, "stitle": "Please Use This Song (Jon Lajoie)", "playlist": null, "title": "Please Use This Song (Jon Lajoie)", "url": "http://vo.fod4.com/v/e402820827/v600.mp4", "extractor_key": "FunnyOrDie", "id": "e402820827", "ext": "mp4", "webpage_url": "http://www.funnyordie.com/videos/e402820827/please-use-this-song-jon-lajoie", "fulltitle": "Please Use This Song (Jon Lajoie)", "thumbnail": null, "webpage_url_basename": "please-use-this-song-jon-lajoie"}

FunnyorDie's RSS feed entry for this page does contain a thumbnail:
media:thumbnail url="http://t.fod4.com/t/e402820827/c480x270_50.jpg" width="464" height="348"

@jillh510
Copy link
Author

@jillh510 jillh510 commented May 6, 2014

Looks like there's an og:image property in the page source:
meta property="og:image" content="http://t.fod4.com/t/e402820827/c1280x720_50.jpg"

@dstftw dstftw closed this in aab8874 May 6, 2014
dstftw added a commit that referenced this issue May 6, 2014
[FunnyOrDie] fix thumbnails + add test (fixes #2856)
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.