[PR #353] [MERGED] server: fix handling of present filters #695

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/353
Author: @nitnelave
Created: 10/26/2022
Status: ✅ Merged
Merged: 10/26/2022
Merged by: @nitnelave

Base: main ← Head: filter

📝 Commits (1)

f0c7313 server: fix handling of present filters

📊 Changes

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

View changed files

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

📄 Description

If the filter name was not in the list of attributes to return, it wouldn't be counted as a valid attribute, meaning that the aliases of attributes were not recognized.

Fixes #351

_{🔄 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/353 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 10/26/2022 **Status:** ✅ Merged **Merged:** 10/26/2022 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `filter` --- ### 📝 Commits (1) - [`f0c7313`](https://github.com/lldap/lldap/commit/f0c7313bcb711c409e20e142c0c5f598e6b344c5) server: fix handling of present filters ### 📊 Changes **3 files changed** (+20 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `server/src/domain/ldap/group.rs` (+1 -1) 📝 `server/src/domain/ldap/user.rs` (+1 -1) 📝 `server/src/infra/ldap_handler.rs` (+18 -0) </details> ### 📄 Description If the filter name was not in the list of attributes to return, it wouldn't be counted as a valid attribute, meaning that the aliases of attributes were not recognized. Fixes #351 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>