Skip to content

Commit

Permalink
fix: compatibility change for edx-drf-extensions 9.0.0
Browse files Browse the repository at this point in the history
edx-drf-extensions 9.0.0 requires VERIFY_LMS_USER_ID_PROPERTY_NAME
to be properly set in LMS to get the appropriate verification when
forgiving JWTs is enabled (which will soon be by default).

See openedx/edx-drf-extensions#408 for details.

This is part of:
edx/edx-arch-experiments#429
  • Loading branch information
robrap committed Nov 27, 2023
1 parent dcdbcd2 commit 18cb734
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions lms/envs/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -4411,6 +4411,8 @@ def _make_locale_paths(settings): # pylint: disable=missing-function-docstring
# Set this value to an empty dict in order to prevent automatically updating
# user data from values in (possibly stale) JWTs.
'JWT_PAYLOAD_USER_ATTRIBUTE_MAPPING': {},
# Allows JWT authentication to find the LMS user id for verification
'VERIFY_LMS_USER_ID_PROPERTY_NAME': 'id',
}

################################ Settings for rss_proxy ################################
Expand Down

0 comments on commit 18cb734

Please sign in to comment.