diff --git a/cms/envs/production.py b/cms/envs/production.py index 012c2912324659ec2890eb55e782cc531c5cc912..bc443b5665bd76f557b73853f7ecb9d299ae517b 100644 --- a/cms/envs/production.py +++ b/cms/envs/production.py @@ -16,8 +16,10 @@ from corsheaders.defaults import default_headers as corsheaders_default_headers from django.core.exceptions import ImproperlyConfigured from django.urls import reverse_lazy from edx_django_utils.plugins import add_plugins +from importlib.metadata import version from path import Path as path + from openedx.core.djangoapps.plugins.constants import ProjectType, SettingsType from .common import * @@ -554,6 +556,13 @@ derive_settings(__name__) if FEATURES.get('ENABLE_CORS_HEADERS'): CORS_ALLOW_CREDENTIALS = True CORS_ORIGIN_WHITELIST = ENV_TOKENS.get('CORS_ORIGIN_WHITELIST', ()) + + # values are already updated above with default CORS_ORIGIN_WHITELIST values but in + # case of new version django_cors_headers they will get override. + cors_major_version = int(version('django_cors_headers').split('.')[0]) + if cors_major_version >= 3 and CORS_ORIGIN_WHITELIST and ENV_TOKENS.get('CORS_ORIGIN_WHITELIST_WITH_SCHEME'): + CORS_ORIGIN_WHITELIST = ENV_TOKENS.get('CORS_ORIGIN_WHITELIST_WITH_SCHEME') + CORS_ORIGIN_ALLOW_ALL = ENV_TOKENS.get('CORS_ORIGIN_ALLOW_ALL', False) CORS_ALLOW_INSECURE = ENV_TOKENS.get('CORS_ALLOW_INSECURE', False) CORS_ALLOW_HEADERS = corsheaders_default_headers + ( diff --git a/lms/djangoapps/experiments/tests/test_views.py b/lms/djangoapps/experiments/tests/test_views.py index b1d6624947300adf5582fdb09f6a0172d327b292..b58c2de7f14fd323ec2b1d997306b2d3bd763cd1 100644 --- a/lms/djangoapps/experiments/tests/test_views.py +++ b/lms/djangoapps/experiments/tests/test_views.py @@ -13,6 +13,7 @@ from django.core.handlers.wsgi import WSGIRequest from django.test.utils import override_settings from django.urls import reverse from django.utils.timezone import now +from importlib_metadata import version from rest_framework.test import APITestCase from common.djangoapps.student.tests.factories import UserFactory @@ -260,6 +261,11 @@ class ExperimentCrossDomainTests(APITestCase): **kwargs ) + def test_white_list_contents_with_cors_header_version(self, *args): # pylint: disable=unused-argument + """ Verify that with django-cor-header<3 it loads list without scheme. """ + assert settings.CORS_ORIGIN_WHITELIST == ['sandbox.edx.org'] + assert int(version('django_cors_headers').split('.')[0]) == 2 + class ExperimentKeyValueViewSetTests(APITestCase): # lint-amnesty, pylint: disable=missing-class-docstring diff --git a/lms/envs/production.py b/lms/envs/production.py index 9ede3006ac3649eb1b9c5e029c422f729f2064ee..1d74e65613f136e55cb7dbc567beb2762c71d3c0 100644 --- a/lms/envs/production.py +++ b/lms/envs/production.py @@ -25,6 +25,7 @@ import yaml from corsheaders.defaults import default_headers as corsheaders_default_headers from django.core.exceptions import ImproperlyConfigured from edx_django_utils.plugins import add_plugins +from importlib.metadata import version from path import Path as path from openedx.core.djangoapps.plugins.constants import ProjectType, SettingsType @@ -356,6 +357,13 @@ CSRF_TRUSTED_ORIGINS = ENV_TOKENS.get('CSRF_TRUSTED_ORIGINS', []) if FEATURES.get('ENABLE_CORS_HEADERS') or FEATURES.get('ENABLE_CROSS_DOMAIN_CSRF_COOKIE'): CORS_ALLOW_CREDENTIALS = True CORS_ORIGIN_WHITELIST = ENV_TOKENS.get('CORS_ORIGIN_WHITELIST', ()) + + # values are already updated above with default CORS_ORIGIN_WHITELIST values but in + # case of new version of django_cors_headers they will get override. + cors_major_version = int(version('django_cors_headers').split('.')[0]) + if cors_major_version >= 3 and CORS_ORIGIN_WHITELIST and ENV_TOKENS.get('CORS_ORIGIN_WHITELIST_WITH_SCHEME'): + CORS_ORIGIN_WHITELIST = ENV_TOKENS.get('CORS_ORIGIN_WHITELIST_WITH_SCHEME') + CORS_ORIGIN_ALLOW_ALL = ENV_TOKENS.get('CORS_ORIGIN_ALLOW_ALL', False) CORS_ALLOW_INSECURE = ENV_TOKENS.get('CORS_ALLOW_INSECURE', False) CORS_ALLOW_HEADERS = corsheaders_default_headers + ( diff --git a/lms/envs/test.py b/lms/envs/test.py index 560aabb11ef0c4f141bfc7283e5284d640680c8d..41f526afcc174df7414b2bf8581e01c6b6a6f069 100644 --- a/lms/envs/test.py +++ b/lms/envs/test.py @@ -20,6 +20,7 @@ from uuid import uuid4 import openid.oidutil from django.utils.translation import ugettext_lazy from edx_django_utils.plugins import add_plugins +from importlib.metadata import version from path import Path as path from openedx.core.djangoapps.plugins.constants import ProjectType, SettingsType @@ -597,3 +598,12 @@ REGISTRATION_RATELIMIT = '5/minute' RESET_PASSWORD_TOKEN_VALIDATE_API_RATELIMIT = '2/m' RESET_PASSWORD_API_RATELIMIT = '2/m' + +CORS_ORIGIN_WHITELIST = ['sandbox.edx.org'] +CORS_ORIGIN_WHITELIST_WITH_SCHEME = ['https://sandbox.edx.org'] + +# values are already updated above with default CORS_ORIGIN_WHITELIST values but in +# case of new version django_cors_headers they will get override. +cors_major_version = int(version('django_cors_headers').split('.')[0]) +if cors_major_version >= 3 and CORS_ORIGIN_WHITELIST and CORS_ORIGIN_WHITELIST_WITH_SCHEME: + CORS_ORIGIN_WHITELIST = CORS_ORIGIN_WHITELIST_WITH_SCHEME