Skip to content

Conversation

le-mikcho
Copy link

TL;DR supports broader LDAP Base Bind DN

This is a dirty hack to automatically resolve the correct userDN in LDAP if other OUs are under the base DN.

For example, your LDAP structure might look like this:
ou=SOMEGROUP01,ou=People,dc=domain,dc=com
ou=SOMEGROUP02,ou=People,dc=domain,dc=com
ou=SOMEGROUP03,ou=People,dc=domain,dc=com

You would be limited to setting the Base Bind DN to a specific OU, such as ou=SOMEGROUP01,ou=People,dc=domain,dc=ch. Users in other groups/OUs under ou=People,dc=domain,dc=com would not be able to log in.

With the fix, all users under the base dn ou=People,dc=domain,dc=com can now log in. sysadmin can now just define a broad base DN and the correct userDN is automagically found ex. uid=username,ou=SOMEGROUP01,ou=People,dc=domain,dc=com in our case.

@le-mikcho le-mikcho requested a review from snipe as a code owner May 23, 2025 12:52
@uberbrady uberbrady self-assigned this May 26, 2025
@snipe snipe requested a review from uberbrady June 4, 2025 11:20
@snipe
Copy link
Member

snipe commented Sep 3, 2025

Hey there - sorry for the delay on reviewing this. We'll try to review it this week. It looks like you're PRing against master though, so you'll need to retarget (likely after a rebase) to develop.

@uberbrady
Copy link
Member

I did a broader take on this here: #17832 - would you be able to check that one out and see if it gets you the same results you were getting here?

@le-mikcho
Copy link
Author

le-mikcho commented Sep 10, 2025

#17832 looks promising ill probably be able to test it by the end of next week. I see I have PR to master, shall i close this one and just create a new PR to develop if #17832 does not help?

@snipe
Copy link
Member

snipe commented Sep 10, 2025

@le-mikcho that would be great, thanks!

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants