[PR #953] [MERGED] server: Fix implementation of attribute present filter #1034

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/953
Author: @nitnelave
Created: 8/16/2024
Status: ✅ Merged
Merged: 8/16/2024
Merged by: @nitnelave

Base: main ← Head: presence

📝 Commits (1)

7db8ee2 server: Fix implementation of attribute present filter

📊 Changes

6 files changed (+72 additions, -18 deletions)

View changed files

📝 server/src/domain/handler.rs (+2 -0)
📝 server/src/domain/ldap/group.rs (+7 -4)
📝 server/src/domain/ldap/user.rs (+7 -7)
📝 server/src/domain/sql_group_backend_handler.rs (+8 -3)
📝 server/src/domain/sql_schema_backend_handler.rs (+40 -1)
📝 server/src/domain/sql_user_backend_handler.rs (+8 -3)

📄 Description

Instead of just doing a schema check, this actually looks for users that have a value for this attribute.

Fixes #950

_{🔄 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/953 **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:** `presence` --- ### 📝 Commits (1) - [`7db8ee2`](https://github.com/lldap/lldap/commit/7db8ee2f65f24c16d0d660075a2637b6a7769d32) server: Fix implementation of attribute present filter ### 📊 Changes **6 files changed** (+72 additions, -18 deletions) <details> <summary>View changed files</summary> 📝 `server/src/domain/handler.rs` (+2 -0) 📝 `server/src/domain/ldap/group.rs` (+7 -4) 📝 `server/src/domain/ldap/user.rs` (+7 -7) 📝 `server/src/domain/sql_group_backend_handler.rs` (+8 -3) 📝 `server/src/domain/sql_schema_backend_handler.rs` (+40 -1) 📝 `server/src/domain/sql_user_backend_handler.rs` (+8 -3) </details> ### 📄 Description Instead of just doing a schema check, this actually looks for users that have a value for this attribute. Fixes #950 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>