[PR #416] [MERGED] Add dn filter support #725

New issue

Closed

opened 2026-02-27 09:09:31 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 09:09:31 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/416
Author: @lucat1
Created: 1/12/2023
Status: ✅ Merged
Merged: 1/17/2023
Merged by: @nitnelave

Base: main ← Head: filter-dn

📝 Commits (4)

aaa6b06 Add dn filter support
398b0d7 tests for dn filter in users and groups
3d2e831 move tests to the ldap_handler level
9b7c388 move tests

📊 Changes

3 files changed (+42 additions, -2 deletions)

View changed files

📝 server/src/domain/ldap/group.rs (+15 -1)
📝 server/src/domain/ldap/user.rs (+17 -1)
📝 server/src/infra/ldap_handler.rs (+10 -0)

📄 Description

Adds support to filter results by the dn name of the resource.

You can try it on the default configuration with:

$ ldapsearch -b "dc=example,dc=com" -D "cn=admin,ou=people,dc=example,dc=com" -W -H ldap://localhost:3890 "(dn=uid=admin,ou=people,dc=example,dc=com)"
$ ldapsearch -b "dc=example,dc=com" -D "cn=admin,ou=people,dc=example,dc=com" -W -H ldap://localhost:3890 "(dn=uid=lldap_strict_readonly,ou=groups,dc=example,dc=com)"

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/lldap/lldap/pull/416 **Author:** [@lucat1](https://github.com/lucat1) **Created:** 1/12/2023 **Status:** ✅ Merged **Merged:** 1/17/2023 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `filter-dn` --- ### 📝 Commits (4) - [`aaa6b06`](https://github.com/lldap/lldap/commit/aaa6b065cc9a0543203164b29a8ae9f4716e9ffb) Add dn filter support - [`398b0d7`](https://github.com/lldap/lldap/commit/398b0d79ef3f2de1e0f22dd3cc72c63a0696679f) tests for dn filter in users and groups - [`3d2e831`](https://github.com/lldap/lldap/commit/3d2e831276dc03dd471b3b9320fb2fa68b6164b5) move tests to the ldap_handler level - [`9b7c388`](https://github.com/lldap/lldap/commit/9b7c38852fc636732d3bda4da9f1bd49926ec6cd) move tests ### 📊 Changes **3 files changed** (+42 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `server/src/domain/ldap/group.rs` (+15 -1) 📝 `server/src/domain/ldap/user.rs` (+17 -1) 📝 `server/src/infra/ldap_handler.rs` (+10 -0) </details> ### 📄 Description Adds support to filter results by the dn name of the resource. You can try it on the default configuration with: ``` $ ldapsearch -b "dc=example,dc=com" -D "cn=admin,ou=people,dc=example,dc=com" -W -H ldap://localhost:3890 "(dn=uid=admin,ou=people,dc=example,dc=com)" $ ldapsearch -b "dc=example,dc=com" -D "cn=admin,ou=people,dc=example,dc=com" -W -H ldap://localhost:3890 "(dn=uid=lldap_strict_readonly,ou=groups,dc=example,dc=com)" ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>