From db8dd8de64842429c19c06a64e48d50aa2e6bc4b Mon Sep 17 00:00:00 2001 From: Greg Price Date: Mon, 10 Nov 2014 18:05:41 -0500 Subject: [PATCH] Fix OAuth token login endpoint to set session user --- common/djangoapps/student/tests/test_login.py | 1 + common/djangoapps/student/views.py | 1 + 2 files changed, 2 insertions(+) diff --git a/common/djangoapps/student/tests/test_login.py b/common/djangoapps/student/tests/test_login.py index b35be740bf..eb3183a4a6 100644 --- a/common/djangoapps/student/tests/test_login.py +++ b/common/djangoapps/student/tests/test_login.py @@ -485,6 +485,7 @@ class LoginOAuthTokenMixin(object): self._setup_user_response(success=True) response = self.client.post(self.url, {"access_token": "dummy"}) self.assertEqual(response.status_code, 204) + self.assertEqual(self.client.session['_auth_user_id'], self.user.id) def test_invalid_token(self): self._setup_user_response(success=False) diff --git a/common/djangoapps/student/views.py b/common/djangoapps/student/views.py index 177c37dc84..0ebf4971dd 100644 --- a/common/djangoapps/student/views.py +++ b/common/djangoapps/student/views.py @@ -1135,6 +1135,7 @@ def login_oauth_token(request, backend): pass # do_auth can return a non-User object if it fails if user and isinstance(user, User): + login(request, user) return JsonResponse(status=204) else: # Ensure user does not re-enter the pipeline