Merge pull request #286 from impak-finance/issue-285

Fixed #285 -- Ensured standard and custom claims can be used at the same time when using the OIDC_IDTOKEN_INCLUDE_CLAIMS setting
This commit is contained in:
Juan Ignacio Fiorentino 2018-09-29 13:24:21 -03:00 committed by GitHub
commit 0577e50b6c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -55,12 +55,11 @@ def create_id_token(token, user, aud, nonce='', at_hash='', request=None, scope=
# Inlude (or not) user standard claims in the id_token.
if settings.get('OIDC_IDTOKEN_INCLUDE_CLAIMS'):
standard_claims = StandardScopeClaims(token)
dic.update(standard_claims.create_response_dic())
if settings.get('OIDC_EXTRA_SCOPE_CLAIMS'):
custom_claims = settings.get('OIDC_EXTRA_SCOPE_CLAIMS', import_str=True)(token)
claims = custom_claims.create_response_dic()
else:
claims = StandardScopeClaims(token).create_response_dic()
dic.update(claims)
dic.update(custom_claims.create_response_dic())
dic = run_processing_hook(
dic, 'OIDC_IDTOKEN_PROCESSING_HOOK',