Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
E
edx-platform
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
edx
edx-platform
Commits
3651f2c0
Commit
3651f2c0
authored
Nov 12, 2014
by
Nimisha Asthagiri
Browse files
Options
Browse Files
Download
Plain Diff
Merge pull request #5921 from edx/gprice/fix-oauth-token-login
Fix OAuth token login endpoint to set session user
parents
257b22e2
db8dd8de
Show whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
2 additions
and
0 deletions
+2
-0
common/djangoapps/student/tests/test_login.py
+1
-0
common/djangoapps/student/views.py
+1
-0
No files found.
common/djangoapps/student/tests/test_login.py
View file @
3651f2c0
...
...
@@ -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
)
...
...
common/djangoapps/student/views.py
View file @
3651f2c0
...
...
@@ -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
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment