hosting/selfhostblocks
1
0
Fork 0
Code Issues Activity

set ldapExpertUsernameAttr to user_id in nextcloud ldap app

Browse source 
https://github.com/lldap/lldap/pull/907
This commit is contained in:
ibizaman 2024-05-01 22:58:15 -07:00 committed by Pierre Penninckx
parent e6b11a5acb
commit d8ba5f97b8
2 changed files with 6 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
CHANGELOG.md
View file

@ -7,6 +7,7 @@
- Remove `shb.vaultwarden.ldapEndpoint` option because it was not used in the implementation anyway. - Remove `shb.vaultwarden.ldapEndpoint` option because it was not used in the implementation anyway.
- Bump Nextcloud default version from 27 to 28. Add support for version 29. - Bump Nextcloud default version from 27 to 28. Add support for version 29.
- Deluge config breaks the authFile into an attrset of user to password file. Also deluge has tests now. - Deluge config breaks the authFile into an attrset of user to password file. Also deluge has tests now.
- Nextcloud now configures the LDAP app to use the `user_id` from LLDAP as the user ID used in Nextcloud. This makes all source of user - internal, LDAP and SSO - agree on the user ID.
## User Facing Backwards Compatible Changes ## User Facing Backwards Compatible Changes

5
modules/services/nextcloud-server.nix
View file

@ -852,6 +852,11 @@ in
'1' '1'
${occ} ldap:set-config "${cID}" 'ldapUserFilterObjectclass' \ ${occ} ldap:set-config "${cID}" 'ldapUserFilterObjectclass' \
'person' 'person'
# Makes the user_id used when creating a user through LDAP which means the ID used in
# Nextcloud is compatible with the one returned by a (possibly added in the future) SSO
# provider.
${occ} ldap:set-config "${cID}" 'ldapExpertUsernameAttr' \
'user_id'
${occ} ldap:test-config -- "${cID}" ${occ} ldap:test-config -- "${cID}"