To fix the corrupt images error, add content.open() to the top of _save() on the S3Storage class (or at least before the chunks() or read() calls). This basically causes a seek(0) on content. I think perhaps some of the image validation reads part of the file and doesn't reset the pointer back to the beginning.
Jay On Sep 1, 10:40 am, Ramdas S <[EMAIL PROTECTED]> wrote: > David, > > I think this is still not fixed. I am getting the same errors > > R > > On Aug 25, 2:02 pm, David Larlet <[EMAIL PROTECTED]> wrote: > > > Le 20 août 08 à 17:56, shadfc a écrit : > > > > With the code from the django-storages you referenced installed > > > somewhere on PYTHONPATH, its as easy as setting a few things in your > > > settings.py. You can see the docs for the code at > > >http://code.larlet.fr/doc/django-s3-storage.html. Put the Required > > > and Optional (if you want it, obviously) stuff in your settings.py and > > > fill in the appropriate values for yourS3account. > > > > Now, assuming those settings are all correct, all of your FileFields > > > (and thus, ImageFields) should store toS3into the bucket you set. > > > You can continue to use upload_to to prefix the filename within the > > > bucket. > > > > Now, I've noticed a few bugs, both of which I've notified David (the > > > author) of: > > > 1) if you call object.filefield.size, this code will download the > > > entire file fromS3just to calculate the size. This happens because > > > of the construction of the _open() method on the S3Storage class. It > > > should not make theS3get call (which downloads the file). The > > > workaround for now is not to use the size property with this code, but > > > the more permanent fix is to delay reading of the file fromS3until > > > read() is called on a S3StorageFile object. > > > 2) I cannot get images to store correctly toS3when using an > > > ImageField. They appear in the bucket with a small filesize change, > > > but no software I have will recognize them as valid images. Storing > > > images (and any other file) via a FileField works just fine and the > > > files are not corrupted. Strangely, storing images via ImageField on > > > the FileSystemStorage backend works just fine, so it only seems to be > > > the combination of an ImageField while using thisS3backend that is > > > the problem. I am not sure where the bug in this is. If you give it > > > a try, see if you can verify this bug for me. > > > > Jay > > > Hi Jay, > > > I plan to work on this tonight because that's clearly a blocking > > point. Thanks for reporting those bugs. > > > Best, > > David --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Django users" group. To post to this group, send email to django-users@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/django-users?hl=en -~----------~----~----~----~------~----~------~--~---
