nextcloud / nextcloud/user_oidc
support flattening of nested groups like keycloak uses
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 181
- Forks
- 60
- Avg merge
- 7h 34m
- Merged PRs (30d)
- 26
Description
How to use GitHub
- Please use the 👍 reaction to show that you are interested into the same feature.
- Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
- Subscribe to receive notifications on status change and new comments.
Feature request
Which Nextcloud Version are you currently using: 29.0.6
Is your feature request related to a problem? Please describe.
In Keycloak it is possible to have nested (hierarchical) groups like "/parent/child". So if a user is member of child group it is implicitly also member of parent group.
Nextcloud doesn't support nested groups.
Unfortunately at the time being keycloak can only send the whole group path "/parent/child" or only the group the user is explicitly member of, so "child'.
In a lot of use cases, having an option which supports flattening the groups would be enough. Like
"/parent/child" becomes the two groups:
- parent
- child
Would be nice if that would be an selectable option for the user_oidc app.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing how the user_oidc app reads group claims from Keycloak and where its user-selectable options are configured. Done means an option can flatten a path such as /parent/child into both parent and child groups, while preserving the existing group behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- authentication, backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100