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 ffbd9f9298..5e5674d1ec 100644 --- a/cms/urls.py +++ b/cms/urls.py @@ -277,12 +277,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 a2666d3d67..d150eac658 100644 --- a/lms/envs/common.py +++ b/lms/envs/common.py @@ -3801,9 +3801,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 @@ -3817,9 +3818,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 d5b6ad8e22..3d48588b06 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.10.1 is major upgrade and require multiple settings changes. +django-storages==1.10.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 5f33f54836..e9455161c8 100644 --- a/requirements/edx/base.txt +++ b/requirements/edx/base.txt @@ -357,7 +357,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.10.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 9910c3fe01..c95115645b 100644 --- a/requirements/edx/development.txt +++ b/requirements/edx/development.txt @@ -570,7 +570,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.10.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 312aa749ba..e74977b70a 100644 --- a/requirements/edx/doc.txt +++ b/requirements/edx/doc.txt @@ -420,7 +420,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.10.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 6f3abc3d97..53e3ae011e 100644 --- a/requirements/edx/testing.txt +++ b/requirements/edx/testing.txt @@ -452,7 +452,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.10.1 # via # -c requirements/edx/../constraints.txt # -r requirements/edx/base.txt