From b098bd921d6910735d5e1dc940b99603ff09980b Mon Sep 17 00:00:00 2001 From: Robert Raposa Date: Thu, 29 Aug 2024 15:40:28 -0400 Subject: [PATCH] feat: DEPR USE-JWT-COOKIE header - Part 1 This repo is no longer using USE-JWT-COOKIE header, since it has the required edx-drf-extensions>10.2.0, where it was fully removed. This removes all uses of the header, except updating CORS_ALLOW_HEADERS, which can't be done before all MFEs and other callers stop sending the header. See "[DEPR]: USE-JWT-COOKIE header" for more details: - https://github.com/openedx/edx-drf-extensions/issues/371 --- openedx/core/djangoapps/user_authn/tests/test_cookies.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/openedx/core/djangoapps/user_authn/tests/test_cookies.py b/openedx/core/djangoapps/user_authn/tests/test_cookies.py index a90f20f19469..8a7841b3b980 100644 --- a/openedx/core/djangoapps/user_authn/tests/test_cookies.py +++ b/openedx/core/djangoapps/user_authn/tests/test_cookies.py @@ -74,9 +74,6 @@ def _copy_cookies_to_request(self, response, request): for key, val in response.cookies.items() } - def _set_use_jwt_cookie_header(self, request): - request.META['HTTP_USE_JWT_COOKIE'] = 'true' - def _assert_recreate_jwt_from_cookies(self, response, can_recreate): """ If can_recreate is True, verifies that a JWT can be properly recreated @@ -133,7 +130,6 @@ def test_set_logged_in_deprecated_cookies(self): @patch.dict("django.conf.settings.FEATURES", {"DISABLE_SET_JWT_COOKIES_FOR_TESTS": False}) def test_set_logged_in_jwt_cookies(self): setup_login_oauth_client() - self._set_use_jwt_cookie_header(self.request) response = cookies_api.set_logged_in_cookies(self.request, HttpResponse(), self.user) self._assert_cookies_present(response, cookies_api.ALL_LOGGED_IN_COOKIE_NAMES) self._assert_consistent_expires(response, num_of_unique_expires=2) @@ -153,7 +149,6 @@ def test_delete_and_are_logged_in_cookies_set(self): @patch.dict("django.conf.settings.FEATURES", {"DISABLE_SET_JWT_COOKIES_FOR_TESTS": False}) def test_refresh_jwt_cookies(self): setup_login_oauth_client() - self._set_use_jwt_cookie_header(self.request) response = cookies_api.get_response_with_refreshed_jwt_cookies(self.request, self.user) data = json.loads(response.content.decode('utf8').replace("'", '"')) assert data['success'] is True