Skip to content

Commit

Permalink
fix: (feedback) pass email from request.user instead of LTI user
Browse files Browse the repository at this point in the history
  • Loading branch information
arslanashraf7 committed May 17, 2024
1 parent 2853676 commit ce92ae0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lms/djangoapps/lti_provider/users.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ def authenticate_lti_user(request, lti_user_id, lti_consumer):
# Verify that the email from the LTI Launch and the logged-in user are the same
# before linking the LtiUser with the edx_user.
if request.user.is_authenticated and request.user.email.lower() == lis_email.lower():
lti_user = create_lti_user(lti_user_id, lti_consumer, lis_email)
lti_user = create_lti_user(lti_user_id, lti_consumer, request.user.email)
else:
# Ask the user to login before linking.
raise PermissionDenied() from exc
Expand Down

0 comments on commit ce92ae0

Please sign in to comment.