ic-labs / django-icekit

73%

develop: 74%

LAST BUILD ON BRANCH add-public-read-s3-storage-for-thumbnails

branch: add-public-read-s3-storage-for-thumbnails

CHANGE BRANCH

x

Reset

• add-public-read-s3-storage-for-thumbnails
• 0.13
• 0.13.1
• 0.14
• 0.14.1
• 0.15
• 0.17
• 0.17.2
• 10-url-override-with-publishing
• 141_thumbnailer_sizes_in_settings
• 176-remove-unpublishable-classes
• 204-iiif-image-api
• 212-api-for-images-collection
• 213-add-collections-api
• 230-split-settings
• 236-site-map-importer
• 241-cache-docker-in-travis
• 247-image-categories-in-api
• 253-travis-failures-installing-docker-engine-1.9
• 254-bulk-delete-of-pages-in-admin
• 261-locations-for-events
• 265-add-mixins-to-simple-event
• 288-reduce-tight-model-relationship-coupling
• 308-fixes-for-fluent-pages-1-1-3
• 49_visually_identify_preview
• 85-rename-preview-link-param
• agsa/master
• develop
• feature/138-fix-admin-content-js-behaviour
• feature/155-show-preview-link-for-staff
• feature/156-simple-content-workflow
• feature/183-test-travis
• feature/184-port-mptt-mgmt-commands
• feature/192-add-republish-permission
• feature/197-backport-templates-to-core
• feature/22-fix-publishing-of-fluent-contents
• feature/286-bower-install-icekit-deps
• feature/286-use-yarn-if-yarn.lock-available
• feature/32-make-publishing-filters-urlnode-compatible
• feature/39_asset_model
• feature/42-fix-publishing-m2m-rels-with-through-table
• feature/44-build-search-features
• feature/45-fix-workflow-app-name
• feature/48-alter-the-nav-model
• feature/add-runtests-quick-docker-env-script
• feature/admin-refactor
• feature/content-listing-plugin
• feature/docker-tag
• feature/dotenv
• feature/fixes-for-icekit-based-projects
• feature/image_improvements
• feature/integrate-doctests-into-test-suite
• feature/media-category-api
• feature/new_articles
• feature/news-aggregator
• feature/publishing-auto-publish
• feature/raw_id_preview
• feature/refactor-location-models
• feature/remove-gulp-dependency
• feature/rest-docs
• feature/reuse-db-tests
• feature/screencast
• feature/travis-docker
• feature/uncompressed_js_block
• feature/urls_in_quotes
• fix-image-gallery-js-ref
• fix-publishing-for-m2ms-with-gfks
• fix-setting-derived-text-plugin-migrations
• glamkit
• glamkit-events
• iiif-improvements
• master
• merge-repos
• news-aggregator
• pinned-startproject
• pinned-startproject-ixc-django-docker
• pre-repo-merge
• public-s3-storage-support
• requires-io-develop
• test-zapier
• tmp-for-aot-prod
• travis-tag-commit
• update-icekit-tests-db

Build # 710

Build Type

push

travis-ci

Committed by jmurty

Commit Message

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

When the `ENABLE_S3_MEDIA` setting is applied, continue to use "private"
S3 storage for media assets by default, but switch to using "public"
S3 storage for thumbnails generated by the thumbnailing system.

Here "private" storage means that images are only accessible through
time-limited signed S3 URLs, while "public" means that thumbnails will
be accessible through non-time-limited standard S3 URLs and the
thumbnail images will be stored with the `public-read` ACL behind the
scenes.

As part of this change, the `S3DefaultStorage` storage class is renamed
to `S3DefaultPrivateStorage` to be explicit (the original class name
remains available for backwards-compatibility) and the new
`S3DefaultPublicStorage` class is added.

Run Details

3934 of 5397 relevant lines covered (72.89%)

0.73 hits per line