Use public-read for S3-enabled storage of thumbnails by default

icekit/project/settings/calculated.py

@@ -126,6 +126,10 @@
 
 # STORAGES ####################################################################
 
+# Override media and thumbnail storage if S3 media is enabled
 if ENABLE_S3_MEDIA:
-    DEFAULT_FILE_STORAGE = 'icekit.utils.storage.S3DefaultStorage'
-    THUMBNAIL_DEFAULT_STORAGE = DEFAULT_FILE_STORAGE
+    # "Private" S3 storage by default for most media assets
+    DEFAULT_FILE_STORAGE = 'icekit.utils.storage.S3DefaultPrivateStorage'
+
+    # "Public" S3 storage by default for thumbnails
+    THUMBNAIL_DEFAULT_STORAGE = 'icekit.utils.storage.S3DefaultPublicStorage'

icekit/utils/storage.py

@@ -54,7 +54,7 @@ def __init__(self, *args, **kwargs):
 # STORAGE CLASSES #############################################################
 
 
-class S3DefaultStorage(
+class S3DefaultPrivateStorage(
         # HashedMediaMixin,
         S3MediaLocationMixin,
         S3PrivateMixin,
@@ -63,4 +63,17 @@ class S3DefaultStorage(
     pass
 
 
+class S3DefaultPublicStorage(
+        # HashedMediaMixin,
+        S3MediaLocationMixin,
+        S3PublicMixin,
+        S3Boto3Storage):
+
+    pass
+
+
+# TODO Deprecated, use `S3DefaultPrivateStorage` instead of `S3DefaultStorage`
+S3DefaultStorage = S3DefaultPrivateStorage
+
+
 default_storage = get_storage_class(settings.DEFAULT_FILE_STORAGE)()