search

splittingred / Gallery

A Gallery add-on for MODx Revolution.
http://svn.modxcms.com/docs/display/ADDON/Gallery
67 stars 61 forks source link

S3 Media Source for storing gallery images #256

Open mtwalsh opened 11 years ago

mtwalsh commented 11 years ago

It would be great to be able to use the Gallery snippet with an S3 media source so images are held on Amazon S3 rather than on the server.

Apologies if this is already possible (I couldn't spot where).

oksushi commented 11 years ago

+1 for this.

Perhaps some part of phpThumbOf could be used for this?

mtwalsh commented 11 years ago

@oksushi Thanks, yes you can serve the images from S3/Cloudfront using phpThumbOf but I need to store the images on S3 as we want to use Gallery on sites that are load-balanced so we can't have anything stored on the server itself other than the source code.

oksushi commented 11 years ago

Agreed — I was referring to reusing code from phpThumbOf, not for using it directly to resize images. Having source images and Gallery-resized images both stored on S3 would be helpful.

mtwalsh commented 11 years ago

@oksushi Ah, apologies. Yes, although I don't think it will even need to interface with S3 itself, if it's just changed to work with an S3 media source hopefully the media source would then handle all of the heavy lifting in terms of get/put onto S3 as that's already built-in.

C8NO28 commented 11 years ago

+1 for this.

UzverNumber47 commented 11 years ago

+1

lithiumlab commented 11 years ago

+1

aksteven commented 11 years ago

Would definitely like to be able to use Gallery with CloudFront/S3. +1

markwillis82 commented 11 years ago

I have just added a pull request with support for media sources for the gallery plugin. This allows S3/cloud files / etc to be used as the storage for images.

We needed to develop this feature to allow us to load balance our modx sites across multiple servers (and not using dirty hacks to rsync images between them)

PR: https://github.com/splittingred/Gallery/pull/269

mtwalsh commented 11 years ago

Great news, thanks Mark.

mtwalsh commented 11 years ago

Does anyone know what the process is to get this pull request integrated and the package updated? Would be good to get it done ASAP.