Merge pull request #589 from safaariman/master

Fix upload issue of SVG thumbnails to S3 storage #587

easy_thumbnails/files.py

@@ -402,6 +402,10 @@ def generate_thumbnail(self, thumbnail_options, silent_template_exception=False)
                 subsampling=subsampling)
         data = img.read()
 
+        # S3 requires the data as bytes.
+        if not isinstance(data, bytes):
+            data = data.encode()
+
         thumbnail = ThumbnailFile(
             filename, file=ContentFile(data), storage=self.thumbnail_storage,
             thumbnail_options=thumbnail_options)

easy_thumbnails/source_generators.py

@@ -44,8 +44,9 @@ def vil_image(source, **options):
 
     if not source:
         return
-    filename = source.source_storage.path(source.file.name)
+    # path method should not be implemented for remote storages. We can pass the file directly.
+    # filename = source.source_storage.path(source.file.name)
     try:
-        return Image.load(filename)
+        return Image.load(source.file)
     except Exception as exc:
         raise exc