-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(LDAP): implement IIsAdmin interface #41650
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
An addition for later for we do not delete groups yet (i.e. there is not cleanup and nothing that dispatches the |
35b843c
to
dc1520c
Compare
Updated the description, that will go into documentation eventually. |
/backport to stable27 |
dc1520c
to
3e861e3
Compare
3e861e3
to
5d351ff
Compare
5d351ff
to
15f8d53
Compare
15f8d53
to
e14a124
Compare
addressed the php-cs test complaint. |
- add configuration to specify one LDAP group acting as admin group (CLI) - implement `isAdmin()` method, basically relying on inGroup against the configured group Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
e14a124
to
37237dc
Compare
/backport to stable28 |
Summary
Promotes an LDAP group (per LDAP configuration) to an admin group.
A group can either be promoted via a dedicates occ call whres the group parameter can be a nextcloud group ID or a group name that will be search against – an exact match is required in that case.:
It is also possible to set the admin group mapping using
occ ldap:set-config $configId ldapAdminGroup $groupId
but as the Nextcloud group ID might not be known (yet) it is especially recommnded for automatized setups to use this command, that would also pull in the group and determine the group ID.In order to demote or reset a promotion, an empty string should be set against to the targeted config's ldapAdminGroup:
Checklist