-
-
Notifications
You must be signed in to change notification settings - Fork 991
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
Feature Request: v.redd.it #72
Comments
v.redd.it videos seem to only be available through HLS/m3u8 and DASH/mpd: gallery-dl is not going to support any video manifests, so unless there is an obvious way to get a direct link to those videos, this is not going to to happen. You could always just use |
Nevermind ... there is apparently a direct link in reddit's JSON response (see this reddit thread), but this wouldn't contain any audio. If this is OK for you, then I'm just going to use this. |
I'd personally prefer to use youtube-dl if it means I get the audio. |
Doing a hella necrobump here, but can this be re-evaulated? |
It seems that this is also still an open issue in youtube-dl, by the way. Issue here: ytdl-org/youtube-dl#14748 So this is probably something that should be addressed if properly fixed upstream. |
Videos from Reddit submissions (e.g. https://www.reddit.com/r/formula1/comments/7qn3db/one_of_my_favourite_slow_motion_images_of_the/) will now get downloaded with the youtube-dl downloader (ae07f92). Should I also add an extractor that recognizes v.redd.it URLs directly and only forwards them to youtube-dl? I initially decided against it, because you can just use youtube-dl itself if you've went through the trouble of getting a v.redd.it URL from a video post ... |
Exactly, I think the Reddit post URLs should be enough. |
Getting the video works great, however the filename seems all messed up, I see in your commit it's trying (I think) to get the ID and title, that does work when having With
Unset:
pure youtube-dl:
tl;dr: breaks with downloader.ytdl.outtmpl set. EDIT: Realised for twitter we get |
You can set
|
Aye, that works great, thanks! Looks like it's time to retire my script then. |
Example link:
https://www.reddit.com/r/formula1/comments/7qn3db/one_of_my_favourite_slow_motion_images_of_the/
in the HTML source of the link above, I find what might be a direct link to the video, but it redirects to the link with the comments and the video.
https://v.redd.it/s0tmbazdv3a01
More examples for unit testing found at https://www.reddit.com/domain/v.redd.it/
The text was updated successfully, but these errors were encountered: