Skip to content

feat(provisioning): Add possibility to use LDAP attributes within provisioning templates - #13500

Merged
kesselb merged 1 commit into
mainfrom
feat/provisioning-ldap-placeholder
Aug 25, 2026
Merged

feat(provisioning): Add possibility to use LDAP attributes within provisioning templates#13500
kesselb merged 1 commit into
mainfrom
feat/provisioning-ldap-placeholder

Conversation

@DerDreschner

Copy link
Copy Markdown
Contributor

At the moment, only the user id and the mail address attached to the account can be used within provisioning templates. This isn't enough for enterprise use-cases where the LDAP may contain attributes that need to be used within the provisioning template.

To enable that use-case, this PR adds a general %LDAP:...% placeholder that allows reading any LDAP attribute associated with the user account. Most often, this may be used for accessing the sAMAccountName via %LDAP:sAMAccountName%, but I've built it without limitations to that attribute for flexibility. In case the read-request fails (e.g., because the attribute doesn't exist), the placeholder isn't being replaced within the template (same behavior like %EMAIL%).

TO-DO

  • Add documentation about accessing LDAP attributes within provisioning templates

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Comment thread lib/Db/Provisioning.php Outdated
…visioning templates

Assisted-by: ClaudeCode:claude-fable-5
Signed-off-by: David Dreschner <david.dreschner@nextcloud.com>
@DerDreschner
DerDreschner force-pushed the feat/provisioning-ldap-placeholder branch from 49c00e5 to a8312c6 Compare August 25, 2026 12:14
@DerDreschner
DerDreschner requested a review from kesselb August 25, 2026 12:15
@kesselb
kesselb enabled auto-merge August 25, 2026 12:19
@kesselb
kesselb merged commit 49492a7 into main Aug 25, 2026
44 checks passed
@kesselb
kesselb deleted the feat/provisioning-ldap-placeholder branch August 25, 2026 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants