ripme icon indicating copy to clipboard operation
ripme copied to clipboard

Add switchable storage backends.

Open bobobo1618 opened this issue 9 years ago • 8 comments

Followup to https://github.com/4pr0n/ripme/issues/285.

This is my (working but horrendously messy) code that adds Google Cloud Storage and Amazon S3 as storage backends for ripme.

bobobo1618 avatar Jan 01 '16 01:01 bobobo1618

And added Google Drive as well.

bobobo1618 avatar Jan 02 '16 02:01 bobobo1618

This is really ambitious!

I checked out the code and could fix some things (like the new RedditRipper changes), but it looks like it broke the VideoRipper -- rips are no longer saved to "videos" subdirectory. I can't find a way to fix this.

4pr0n avatar Apr 17 '16 13:04 4pr0n

Do they get saved anywhere else? Admittedly I don't have any videos to rip. I messed around a lot with the directory naming and stuff (AbstractRipper) so I suspect they might be missing the prefix.

bobobo1618 avatar Apr 17 '16 16:04 bobobo1618

Videos are saved to the root folder (where ripme.jar is, one dir above rips).

I tried inserting videos into the path (in VideoRipper.java) but it tried to save files to {full_path_to_ripme.jar} / {full_path_to_videos_dir}

Here's a SFW video you can test the ripper with: https://vid.me/4ahQ

4pr0n avatar Apr 17 '16 19:04 4pr0n

Ah, I remember now.

The issue was that I just didn't port the DownloadVideoThread over to the new infrastructure. It seems to work now but looking through it it's likely I've left a few little bugs due to my lack of understanding the code.

bobobo1618 avatar Apr 17 '16 21:04 bobobo1618

I'm interested in this but it's a significant change so I'm not going to prioritize this for the moment. In the meantime, any work you could put in to test it and clean it up would be appreciated!

metaprime avatar Dec 31 '16 22:12 metaprime

@bobobo1618 are you still interested in getting this PR merged? If so make it in https://github.com/RipMeApp/ripme as development has moved there

cyian-1756 avatar Nov 02 '17 16:11 cyian-1756

@cyian-1756 Ultimately I think we need to decide whether we want to take this, and if so, we need to adopt the code and fix it up ourselves (port it over to ripmeapp/ripme)

metaprime avatar Nov 21 '17 11:11 metaprime