Skip to content

Commit

Permalink
Merge pull request #8853 from readthedocs/humitos/upgrade-django-stor…
Browse files Browse the repository at this point in the history
…ages

django-storages: upgrade
  • Loading branch information
humitos committed Jan 27, 2022
2 parents 2377094 + af217d0 commit e7102a4
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 24 deletions.
9 changes: 0 additions & 9 deletions docs/dev/install.rst
Original file line number Diff line number Diff line change
Expand Up @@ -62,15 +62,6 @@ Set up your environment

inv docker.up --init # --init is only needed the first time

#. add read permissions to the storage backend:

* go to http://localhost:9000/ (MinIO S3 storage backend)
* login as ``admin`` / ``password``
* click "..." next to the ``static`` bucket name and then "Edit Policy"
* leave "prefix" empty and click "Add" to give "Read Only" access on the ``static`` bucket
* click on the "+" icon on the bottom-right corner, then "Create bucket" with the name ``media``,
hit Enter on the keyboard, and repeat the operation above to give "Read Only" access to it

#. go to http://community.dev.readthedocs.io to access your local instance of Read the Docs.


Expand Down
3 changes: 0 additions & 3 deletions readthedocs/settings/docker_compose.py
Original file line number Diff line number Diff line change
Expand Up @@ -167,9 +167,6 @@ def show_debug_toolbar(request):
S3_STATIC_STORAGE_OVERRIDE_HOSTNAME = 'community.dev.readthedocs.io'
S3_MEDIA_STORAGE_OVERRIDE_HOSTNAME = 'community.dev.readthedocs.io'

AWS_AUTO_CREATE_BUCKET = True
AWS_DEFAULT_ACL = 'public-read'
AWS_BUCKET_ACL = 'public-read'
AWS_S3_ENCRYPTION = False
AWS_S3_SECURE_URLS = False
AWS_S3_USE_SSL = False
Expand Down
13 changes: 1 addition & 12 deletions requirements/pip.txt
Original file line number Diff line number Diff line change
Expand Up @@ -101,18 +101,7 @@ django-cors-middleware==1.4.0 # pyup: ignore
# User agent parsing - used for analytics purposes
user-agents==2.2.0


# Utilities used to upload build media to cloud storage
# django-storages is pinned to this particular commit because it
# supports generating URLs with other method than GET when using
# private buckets.
#
# Besides, support for the corresponding AWS_BUCKET_ACL and
# AWS_AUTO_CREATE_BUCKET settings have been removed in 1.10. We depend on this
# in our Docker setup. We can upgrade it but we need to add a
# `create_buckets.sh` to be called on `--init` as we used to do for Azurite
# https://github.com/jschneier/django-storages/pull/636
git+https://github.com/jschneier/django-storages@d0f027c98a877f75615cfc42b4d51c038fa41bf6#egg=django-storages[boto3]==1.9.1
django-storages==1.12.3


# Required only in development and linting
Expand Down

0 comments on commit e7102a4

Please sign in to comment.