Skip to content

Commit 56a8da3

Browse files
committed
Fix RemovedInDjango40Warning for middleware get_resopnse()
Fixes #7417. Fixes all these issues seen with `tox -e py38-django31`: ``` /Users/chainz/Documents/Projects/django-rest-framework/tests/test_request.py:208: RemovedInDjango40Warning: Passing None for the middleware get_response argument is deprecated. SessionMiddleware().process_request(self.wrapped_request) tests/test_requests_client.py: 1 test with warning tests/test_testing.py: 4 tests with warnings tests/test_throttling.py: 1 test with warning tests/authentication/test_authentication.py: 4 tests with warnings tests/browsable_api/test_browsable_api.py: 4 tests with warnings /Users/chainz/Documents/Projects/django-rest-framework/rest_framework/authentication.py:139: RemovedInDjango40Warning: Passing None for the middleware get_response argument is deprecated. check = CSRFCheck() ```
1 parent 327cbef commit 56a8da3

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

rest_framework/authentication.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,10 @@ def enforce_csrf(self, request):
136136
"""
137137
Enforce CSRF validation for session based authentication.
138138
"""
139-
check = CSRFCheck()
139+
def dummy_get_response(request): # pragma: no cover
140+
return None
141+
142+
check = CSRFCheck(dummy_get_response)
140143
# populates request.META['CSRF_COOKIE'], which is used in process_view()
141144
check.process_request(request)
142145
reason = check.process_view(request, None, (), {})

tests/test_request.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -205,8 +205,12 @@ def setUp(self):
205205
# available to login and logout functions
206206
self.wrapped_request = factory.get('/')
207207
self.request = Request(self.wrapped_request)
208-
SessionMiddleware().process_request(self.wrapped_request)
209-
AuthenticationMiddleware().process_request(self.wrapped_request)
208+
209+
def dummy_get_response(request): # pragma: no cover
210+
return None
211+
212+
SessionMiddleware(dummy_get_response).process_request(self.wrapped_request)
213+
AuthenticationMiddleware(dummy_get_response).process_request(self.wrapped_request)
210214

211215
User.objects.create_user('ringo', 'starr@thebeatles.com', 'yellow')
212216
self.user = authenticate(username='ringo', password='yellow')

0 commit comments

Comments
 (0)