Skip to content

LDAP: mention FreeIPA in the distinguished-name group member option - #1122

Merged
tobydox merged 1 commit into
veyon:mainfrom
rautyrauty:ldap-mention-freeipa
May 22, 2026
Merged

tobydox merged 1 commit into
veyon:mainfrom
rautyrauty:ldap-mention-freeipa

Conversation

@rautyrauty

Copy link
Copy Markdown
Contributor

FreeIPA (389 Directory Server) stores group membership in the member attribute as distinguished names, exactly like Samba/AD — not as login names like a plain OpenLDAP setup.

The "Group member identification" radio button is labelled "Distinguished name (Samba/AD)", so FreeIPA administrators tend to pick the other option ("... user login name ... (OpenLDAP)") instead. With that scheme the groups of user query builds the filter member=<login>, which never matches the DN values stored by FreeIPA, and group-based access control silently stops working.

This was reported downstream as ALT bug #51651: the LDAP integration test fails until the user switches group member identification to "Distinguished name".

This change adds FreeIPA to the label so the correct scheme is obvious. UI-string-only change; translations are left to be picked up by the usual lupdate / Weblate workflow.

FreeIPA (389 Directory Server) stores group membership in the "member"
attribute as distinguished names, exactly like Samba/AD - not as login
names like a plain OpenLDAP setup.

The "Group member identification" radio button is labelled
"Distinguished name (Samba/AD)", so FreeIPA administrators tend to pick
the other option ("... user login name ... (OpenLDAP)") instead. With
that scheme the "groups of user" query builds the filter
"member=<login>", which never matches the DN values stored by FreeIPA,
and group-based access control silently stops working.

Add FreeIPA to the label so the correct scheme is obvious.
@tobydox
tobydox merged commit dd5787c into veyon:main May 22, 2026
@tobydox

tobydox commented May 22, 2026

Copy link
Copy Markdown
Member

Thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants