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

add support for new instagramm videos #904

Closed
ghost opened this issue Jun 22, 2013 · 7 comments
Closed

add support for new instagramm videos #904

ghost opened this issue Jun 22, 2013 · 7 comments

Comments

@ghost
Copy link

@ghost ghost commented Jun 22, 2013

thank you very much for Vines support. Now Facebook/Instagram added also video.

example URL

youtube-dl http://statigr.am/p/483353566818108572_46576824
WARNING: Falling back on generic information extractor.
[generic] 483353566818108572_46576824: Downloading webpage
[generic] 483353566818108572_46576824: Extracting information
ERROR: Invalid URL: http://statigr.am/p/483353566818108572_46576824; please report this issue on GitHub.

@yasoob
Copy link
Contributor

@yasoob yasoob commented Jun 23, 2013

Ah its very easy to make an IE for statigr.am The url of the video file is directly embedded in the page. :) Let me see.

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Jun 23, 2013

@yasoob I have added support for sites that use Open Graph (like statigr.am) to the GenericIE, but feel free to write a complete IE for the site.

@yasoob
Copy link
Contributor

@yasoob yasoob commented Jun 23, 2013

@jaimeMF Then why does youtube-dl give error on this url ? :/ I tested another url and still the same error.

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Jun 23, 2013

@yasoob Have you update to the last commit?, I have just committed it a few minutes ago
Also post the url and I will test.

@yasoob
Copy link
Contributor

@yasoob yasoob commented Jun 23, 2013

Yes it works now. I just tested it again . Thanx and BTW i have done a pull request with a separate IE and test for statigram videos.

@phihag
Copy link
Contributor

@phihag phihag commented Jun 23, 2013

Will work with youtube-dl 2013.06.23 and newer.

@phihag phihag closed this Jun 23, 2013
@ghost
Copy link
Author

@ghost ghost commented Jul 1, 2013

thank you guys. http://statigr.am works fine, but instagram.com uses generic info extractor.

I love title in file name ;)

youtube-dl 'http://instagram.com/p/bNOFljP0Sq/#'
[redirect] Following redirect to http://instagram.com/p/bNOFljP0Sq/
WARNING: Falling back on generic information extractor.
[generic] : Downloading webpage
[generic] : Extracting information
[download] Destination: video-d54b648ce1f511e29e9622000a9f09f1_101.mp4
[download] 100.0% of 2.42MiB at 1.08MiB/s ETA 00:00

youtube-dl 'http://instagram.com/p/bKvbmkMSBL/#'
[redirect] Following redirect to http://instagram.com/p/bKvbmkMSBL/
WARNING: Falling back on generic information extractor.
[generic] : Downloading webpage
[generic] : Extracting information
[download] Destination: video-294b477ce13411e29b7022000a1fbd93_101.mp4
[download] 100.0% of 1.50MiB at 548.75KiB/s ETA 00:00

jaimeMF added a commit that referenced this issue Jul 1, 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
3 participants
You can’t perform that action at this time.