Skip to content
Snippets Groups Projects
Commit eabf097d authored by Nik | Klampfradler's avatar Nik | Klampfradler
Browse files

Merge branch '510--openid-connect-group-claim' into 'master'

Resolve "[OpenID Connect] Group claim"

Closes #510

See merge request !718
parents 35943230 dd81c4a6
No related branches found
No related tags found
1 merge request!718Resolve "[OpenID Connect] Group claim"
Pipeline #34146 canceled
...@@ -12,6 +12,8 @@ Unreleased ...@@ -12,6 +12,8 @@ Unreleased
Added Added
~~~~~ ~~~~~
* OpenID connect scope and accompanying claim `groups`
Fixed Fixed
~~~~~ ~~~~~
......
...@@ -345,6 +345,7 @@ if _settings.get("oauth2.oidc.enabled", False): ...@@ -345,6 +345,7 @@ if _settings.get("oauth2.oidc.enabled", False):
"address": _("Full home postal address"), "address": _("Full home postal address"),
"email": _("Email address"), "email": _("Email address"),
"phone": _("Home and mobile phone"), "phone": _("Home and mobile phone"),
"groups": _("Groups"),
} }
) )
......
...@@ -77,6 +77,9 @@ class CustomOAuth2Validator(OAuth2Validator): ...@@ -77,6 +77,9 @@ class CustomOAuth2Validator(OAuth2Validator):
"postal_code": request.user.person.postal_code, "postal_code": request.user.person.postal_code,
} }
if "groups" in request.scopes and has_person(request.user):
claims["groups"] = request.user.person.groups.values_list("name", flat=True).all()
return claims return claims
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment