diff --git a/cms/djangoapps/contentstore/tests/test_video_utils.py b/cms/djangoapps/contentstore/tests/test_video_utils.py index 103c8179a7..5ba7384dba 100644 --- a/cms/djangoapps/contentstore/tests/test_video_utils.py +++ b/cms/djangoapps/contentstore/tests/test_video_utils.py @@ -389,7 +389,7 @@ class S3Boto3TestCase(TestCase): @override_settings(VIDEO_IMAGE_SETTINGS={ 'STORAGE_CLASS': 'storages.backends.s3boto3.S3Boto3Storage', 'STORAGE_KWARGS': - {'bucket_name': 'test', 'default_acl': None, 'base_url': '/', 'location': 'abc/def'}} + {'bucket_name': 'test', 'default_acl': None, 'location': 'abc/def'}} ) def test_boto3_backend_with_params(self): storage = get_storage_class( @@ -417,7 +417,6 @@ class S3Boto3TestCase(TestCase): obj.upload_fileobj.assert_called_with( content, ExtraArgs={ - 'ACL': 'public-read', # it will come from 1.9.1 'ContentType': 'text/plain', } ) diff --git a/cms/envs/common.py b/cms/envs/common.py index e799613371..a0ae0f0054 100644 --- a/cms/envs/common.py +++ b/cms/envs/common.py @@ -2504,9 +2504,9 @@ VIDEO_IMAGE_SETTINGS = dict( # STORAGE_KWARGS=dict(bucket='video-image-bucket'), STORAGE_KWARGS=dict( location=MEDIA_ROOT, - base_url=MEDIA_URL, ), DIRECTORY_PREFIX='video-images/', + BASE_URL=MEDIA_URL, ) VIDEO_IMAGE_MAX_AGE = 31536000 @@ -2519,9 +2519,9 @@ VIDEO_TRANSCRIPTS_SETTINGS = dict( # STORAGE_KWARGS=dict(bucket='video-transcripts-bucket'), STORAGE_KWARGS=dict( location=MEDIA_ROOT, - base_url=MEDIA_URL, ), DIRECTORY_PREFIX='video-transcripts/', + BASE_URL=MEDIA_URL, ) VIDEO_TRANSCRIPTS_MAX_AGE = 31536000 diff --git a/cms/envs/devstack-experimental.yml b/cms/envs/devstack-experimental.yml index 08cb101f5f..61a94c4bc5 100644 --- a/cms/envs/devstack-experimental.yml +++ b/cms/envs/devstack-experimental.yml @@ -512,17 +512,17 @@ VIDEO_IMAGE_MAX_AGE: 31536000 VIDEO_IMAGE_SETTINGS: DIRECTORY_PREFIX: video-images/ STORAGE_KWARGS: - base_url: /media/ location: /edx/var/edxapp/media// VIDEO_IMAGE_MAX_BYTES: 2097152 VIDEO_IMAGE_MIN_BYTES: 2048 + BASE_URL: /media/ VIDEO_TRANSCRIPTS_MAX_AGE: 31536000 VIDEO_TRANSCRIPTS_SETTINGS: DIRECTORY_PREFIX: video-transcripts/ STORAGE_KWARGS: - base_url: /media/ - location: /edx/var/edxapp/media// + location: edx/var/edxapp/media// VIDEO_TRANSCRIPTS_MAX_BYTES: 3145728 + BASE_URL: /media/ VIDEO_UPLOAD_PIPELINE: BUCKET: '' ROOT_PATH: '' diff --git a/cms/envs/test.py b/cms/envs/test.py index 118b3f1b0c..ebaaf09171 100644 --- a/cms/envs/test.py +++ b/cms/envs/test.py @@ -303,9 +303,9 @@ VIDEO_IMAGE_SETTINGS = dict( VIDEO_IMAGE_MIN_BYTES=2 * 1024, # 2 KB STORAGE_KWARGS=dict( location=MEDIA_ROOT, - base_url=MEDIA_URL, ), DIRECTORY_PREFIX='video-images/', + BASE_URL=MEDIA_URL, ) VIDEO_IMAGE_DEFAULT_FILENAME = 'default_video_image.png' diff --git a/cms/urls.py b/cms/urls.py index 0702128ab6..aae5aeefbd 100644 --- a/cms/urls.py +++ b/cms/urls.py @@ -280,12 +280,12 @@ if settings.DEBUG: urlpatterns += static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT) urlpatterns += static( - settings.VIDEO_IMAGE_SETTINGS['STORAGE_KWARGS']['base_url'], + settings.VIDEO_IMAGE_SETTINGS['BASE_URL'], document_root=settings.VIDEO_IMAGE_SETTINGS['STORAGE_KWARGS']['location'] ) urlpatterns += static( - settings.VIDEO_TRANSCRIPTS_SETTINGS['STORAGE_KWARGS']['base_url'], + settings.VIDEO_TRANSCRIPTS_SETTINGS['BASE_URL'], document_root=settings.VIDEO_TRANSCRIPTS_SETTINGS['STORAGE_KWARGS']['location'] ) diff --git a/lms/envs/common.py b/lms/envs/common.py index 78e35d3e3f..a7c2603030 100644 --- a/lms/envs/common.py +++ b/lms/envs/common.py @@ -3802,9 +3802,10 @@ VIDEO_IMAGE_SETTINGS = dict( # STORAGE_KWARGS=dict(bucket='video-image-bucket'), STORAGE_KWARGS=dict( location=MEDIA_ROOT, - base_url=MEDIA_URL, ), DIRECTORY_PREFIX='video-images/', + BASE_URL=MEDIA_URL, + ) VIDEO_IMAGE_MAX_AGE = 31536000 @@ -3818,9 +3819,9 @@ VIDEO_TRANSCRIPTS_SETTINGS = dict( # STORAGE_KWARGS=dict(bucket='video-transcripts-bucket'), STORAGE_KWARGS=dict( location=MEDIA_ROOT, - base_url=MEDIA_URL, ), DIRECTORY_PREFIX='video-transcripts/', + BASE_URL=MEDIA_URL, ) VIDEO_TRANSCRIPTS_MAX_AGE = 31536000 diff --git a/lms/envs/devstack-experimental.yml b/lms/envs/devstack-experimental.yml index e69656a803..17dadd7e1a 100644 --- a/lms/envs/devstack-experimental.yml +++ b/lms/envs/devstack-experimental.yml @@ -614,17 +614,17 @@ VIDEO_IMAGE_MAX_AGE: 31536000 VIDEO_IMAGE_SETTINGS: DIRECTORY_PREFIX: video-images/ STORAGE_KWARGS: - base_url: /media/ - location: /edx/var/edxapp/media// + location: edx/var/edxapp/media// VIDEO_IMAGE_MAX_BYTES: 2097152 VIDEO_IMAGE_MIN_BYTES: 2048 + BASE_URL: /media/ VIDEO_TRANSCRIPTS_MAX_AGE: 31536000 VIDEO_TRANSCRIPTS_SETTINGS: DIRECTORY_PREFIX: video-transcripts/ STORAGE_KWARGS: - base_url: /media/ - location: /edx/var/edxapp/media// + location: edx/var/edxapp/media// VIDEO_TRANSCRIPTS_MAX_BYTES: 3145728 + BASE_URL: /media/ VIDEO_UPLOAD_PIPELINE: BUCKET: '' ROOT_PATH: '' diff --git a/requirements/constraints.txt b/requirements/constraints.txt index 7feb09c5d4..09c2df98f4 100644 --- a/requirements/constraints.txt +++ b/requirements/constraints.txt @@ -20,9 +20,8 @@ celery>=5.2.2,<6.0.0 # required for celery>=5.2.0;<5.3.0 click>=8.0,<9.0 -# django-storages version 1.9 drops support for boto storage backend. -# 1.9 gives an error for details https://github.com/jschneier/django-storages/issues/831 -django-storages==1.9.1 +# django-storages version 1.11.1 is major version upgrade. +django-storages==1.11.1 # The team that owns this package will manually bump this package rather than having it pulled in automatically. diff --git a/requirements/edx/base.txt b/requirements/edx/base.txt index c7e395ca7b..8a0d605c7e 100644 --- a/requirements/edx/base.txt +++ b/requirements/edx/base.txt @@ -358,7 +358,7 @@ django-statici18n==2.4.0 # -r requirements/edx/kernel.in # lti-consumer-xblock # xblock-drag-and-drop-v2 -django-storages==1.9.1 +django-storages==1.11.1 # via # -c requirements/edx/../constraints.txt # -r requirements/edx/kernel.in diff --git a/requirements/edx/development.txt b/requirements/edx/development.txt index cfec5fb61a..5e3d2b5fa9 100644 --- a/requirements/edx/development.txt +++ b/requirements/edx/development.txt @@ -571,7 +571,7 @@ django-statici18n==2.4.0 # -r requirements/edx/testing.txt # lti-consumer-xblock # xblock-drag-and-drop-v2 -django-storages==1.9.1 +django-storages==1.11.1 # via # -c requirements/edx/../constraints.txt # -r requirements/edx/doc.txt diff --git a/requirements/edx/doc.txt b/requirements/edx/doc.txt index bcdf24b8c6..70b284a920 100644 --- a/requirements/edx/doc.txt +++ b/requirements/edx/doc.txt @@ -421,7 +421,7 @@ django-statici18n==2.4.0 # -r requirements/edx/base.txt # lti-consumer-xblock # xblock-drag-and-drop-v2 -django-storages==1.9.1 +django-storages==1.11.1 # via # -c requirements/edx/../constraints.txt # -r requirements/edx/base.txt diff --git a/requirements/edx/testing.txt b/requirements/edx/testing.txt index 8e68af483e..bedea41d48 100644 --- a/requirements/edx/testing.txt +++ b/requirements/edx/testing.txt @@ -453,7 +453,7 @@ django-statici18n==2.4.0 # -r requirements/edx/base.txt # lti-consumer-xblock # xblock-drag-and-drop-v2 -django-storages==1.9.1 +django-storages==1.11.1 # via # -c requirements/edx/../constraints.txt # -r requirements/edx/base.txt