[PR #951] [MERGED] server: Add support for memberOf with plain user names, relax hard errors #1033

New issue

Closed

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

kerem commented

2026-02-27 09:10:40 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/951
Author: @nitnelave
Created: 8/16/2024
Status: ✅ Merged
Merged: 8/16/2024
Merged by: @nitnelave

Base: main ← Head: filter_error

📝 Commits (1)

a63f0e5 server: Add support for memberOf with plain user names, relax hard errors

📊 Changes

4 files changed (+93 additions, -52 deletions)

View changed files

📝 server/src/domain/ldap/group.rs (+28 -24)
📝 server/src/domain/ldap/user.rs (+23 -18)
📝 server/src/domain/ldap/utils.rs (+28 -0)
📝 server/src/infra/ldap_handler.rs (+14 -10)

📄 Description

This should help when the client sends some invalid-looking queries as part of a bigger filter

Solves #949

_{🔄 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/951 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 8/16/2024 **Status:** ✅ Merged **Merged:** 8/16/2024 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `filter_error` --- ### 📝 Commits (1) - [`a63f0e5`](https://github.com/lldap/lldap/commit/a63f0e5980d4b2b262dea37af449adeaaa2ff1bb) server: Add support for memberOf with plain user names, relax hard errors ### 📊 Changes **4 files changed** (+93 additions, -52 deletions) <details> <summary>View changed files</summary> 📝 `server/src/domain/ldap/group.rs` (+28 -24) 📝 `server/src/domain/ldap/user.rs` (+23 -18) 📝 `server/src/domain/ldap/utils.rs` (+28 -0) 📝 `server/src/infra/ldap_handler.rs` (+14 -10) </details> ### 📄 Description This should help when the client sends some invalid-looking queries as part of a bigger filter Solves #949 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>