[PR #1113] [MERGED] domain-handlers: move backend handler traits to separate crate #1127

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/1113
Author: @broeng
Created: 2/21/2025
Status: ✅ Merged
Merged: 2/21/2025
Merged by: @nitnelave

Base: main ← Head: domain-handlers-crate

📝 Commits (1)

69d398d domain-handlers: move backend handler traits to separate crate

📊 Changes

25 files changed (+149 additions, -104 deletions)

View changed files

📝 Cargo.lock (+19 -0)
📝 Cargo.toml (+1 -0)
➕ crates/domain-handlers/Cargo.toml (+41 -0)
📝 crates/domain-handlers/src/handler.rs (+17 -0)
➕ crates/domain-handlers/src/lib.rs (+1 -0)
📝 server/Cargo.toml (+3 -0)
📝 server/src/domain/ldap/group.rs (+1 -1)
📝 server/src/domain/ldap/user.rs (+1 -1)
📝 server/src/domain/ldap/utils.rs (+1 -18)
📝 server/src/domain/mod.rs (+0 -1)
📝 server/src/domain/sql_backend_handler.rs (+7 -12)
📝 server/src/domain/sql_group_backend_handler.rs (+6 -8)
📝 server/src/domain/sql_opaque_handler.rs (+1 -1)
📝 server/src/domain/sql_schema_backend_handler.rs (+4 -8)
📝 server/src/domain/sql_user_backend_handler.rs (+6 -7)
📝 server/src/infra/access_control.rs (+6 -7)
📝 server/src/infra/auth_service.rs (+4 -4)
📝 server/src/infra/graphql/api.rs (+9 -11)
📝 server/src/infra/graphql/mutation.rs (+2 -3)
📝 server/src/infra/graphql/query.rs (+2 -2)

...and 5 more files

📄 Description

Move traits related to backend handler to separate crate.

_{🔄 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/1113 **Author:** [@broeng](https://github.com/broeng) **Created:** 2/21/2025 **Status:** ✅ Merged **Merged:** 2/21/2025 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `domain-handlers-crate` --- ### 📝 Commits (1) - [`69d398d`](https://github.com/lldap/lldap/commit/69d398df5a62b222a873aed043e1ebf81b8277b2) domain-handlers: move backend handler traits to separate crate ### 📊 Changes **25 files changed** (+149 additions, -104 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+19 -0) 📝 `Cargo.toml` (+1 -0) ➕ `crates/domain-handlers/Cargo.toml` (+41 -0) 📝 `crates/domain-handlers/src/handler.rs` (+17 -0) ➕ `crates/domain-handlers/src/lib.rs` (+1 -0) 📝 `server/Cargo.toml` (+3 -0) 📝 `server/src/domain/ldap/group.rs` (+1 -1) 📝 `server/src/domain/ldap/user.rs` (+1 -1) 📝 `server/src/domain/ldap/utils.rs` (+1 -18) 📝 `server/src/domain/mod.rs` (+0 -1) 📝 `server/src/domain/sql_backend_handler.rs` (+7 -12) 📝 `server/src/domain/sql_group_backend_handler.rs` (+6 -8) 📝 `server/src/domain/sql_opaque_handler.rs` (+1 -1) 📝 `server/src/domain/sql_schema_backend_handler.rs` (+4 -8) 📝 `server/src/domain/sql_user_backend_handler.rs` (+6 -7) 📝 `server/src/infra/access_control.rs` (+6 -7) 📝 `server/src/infra/auth_service.rs` (+4 -4) 📝 `server/src/infra/graphql/api.rs` (+9 -11) 📝 `server/src/infra/graphql/mutation.rs` (+2 -3) 📝 `server/src/infra/graphql/query.rs` (+2 -2) _...and 5 more files_ </details> ### 📄 Description Move traits related to backend handler to separate crate. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>