[PR #1152] [MERGED] Explode the server crate into many small crates #1150

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/1152
Author: @nitnelave
Created: 4/9/2025
Status: ✅ Merged
Merged: 4/9/2025
Merged by: @nitnelave

Base: main ← Head: ldap_crate

📝 Commits (10+)

f1fea83 server: split off ldap/search from ldap_handler
96cae40 server: split off do_bind from ldap_handler
4f7d024 server: split off password handling from ldap_handler
52f3b40 server: split off compare from ldap_handler
f5c2de9 server: split off create_* from ldap_handler
dbd0906 server: split off modify requests from ldap_handler
2540fe7 server: add tests for ldap modify
736d168 server: Add support for deleting users and groups via LDAP
a56f82b server: cleanup extra mut in ldap handler
37926a4 server: Move PublicSchema to the domain crate

📊 Changes

63 files changed (+4864 additions, -3664 deletions)

View changed files

📝 Cargo.lock (+129 -7)
➕ crates/access-control/Cargo.toml (+26 -0)
📝 crates/access-control/src/lib.rs (+13 -11)
📝 crates/auth/Cargo.toml (+1 -1)
📝 crates/auth/src/opaque.rs (+6 -0)
📝 crates/domain-model/Cargo.toml (+2 -8)
📝 crates/domain/src/deserialize.rs (+1 -1)
📝 crates/domain/src/lib.rs (+2 -0)
📝 crates/domain/src/public_schema.rs (+1 -1)
➕ crates/graphql-server/Cargo.toml (+75 -0)
➕ crates/graphql-server/src/api.rs (+91 -0)
📝 crates/graphql-server/src/lib.rs (+0 -0)
📝 crates/graphql-server/src/mutation.rs (+16 -14)
📝 crates/graphql-server/src/query.rs (+15 -16)
➕ crates/ldap/Cargo.toml (+56 -0)
➕ crates/ldap/src/compare.rs (+240 -0)
📝 crates/ldap/src/core/error.rs (+0 -0)
📝 crates/ldap/src/core/group.rs (+12 -15)
📝 crates/ldap/src/core/mod.rs (+0 -0)
📝 crates/ldap/src/core/user.rs (+13 -14)

...and 43 more files

📄 Description

This introduces many small crates, clarifying the dependencies and cleaning up the code.

_{🔄 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/1152 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 4/9/2025 **Status:** ✅ Merged **Merged:** 4/9/2025 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `ldap_crate` --- ### 📝 Commits (10+) - [`f1fea83`](https://github.com/lldap/lldap/commit/f1fea83aa957b7538dbefbd7038d129a4917a987) server: split off ldap/search from ldap_handler - [`96cae40`](https://github.com/lldap/lldap/commit/96cae4001c785d9e9f297760fc81447340203741) server: split off do_bind from ldap_handler - [`4f7d024`](https://github.com/lldap/lldap/commit/4f7d024ffdf1234aab22fb9eb12f034f41cf14a4) server: split off password handling from ldap_handler - [`52f3b40`](https://github.com/lldap/lldap/commit/52f3b40626f113c1f56b2816acf8c647a6d13f64) server: split off compare from ldap_handler - [`f5c2de9`](https://github.com/lldap/lldap/commit/f5c2de91d46fd3c47c023d1e0ca15fe881c51829) server: split off create_* from ldap_handler - [`dbd0906`](https://github.com/lldap/lldap/commit/dbd090645a850824233646145db4061d051400c0) server: split off modify requests from ldap_handler - [`2540fe7`](https://github.com/lldap/lldap/commit/2540fe73d116ac5e15c1209b6c17d7a6fcde008b) server: add tests for ldap modify - [`736d168`](https://github.com/lldap/lldap/commit/736d16861d542ac6e21e39abcb70dc99d0954768) server: Add support for deleting users and groups via LDAP - [`a56f82b`](https://github.com/lldap/lldap/commit/a56f82beeec9381a6be680f799ce91ab30841931) server: cleanup extra mut in ldap handler - [`37926a4`](https://github.com/lldap/lldap/commit/37926a45bd947da57d765a8905d6d7c63d7361fc) server: Move PublicSchema to the domain crate ### 📊 Changes **63 files changed** (+4864 additions, -3664 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+129 -7) ➕ `crates/access-control/Cargo.toml` (+26 -0) 📝 `crates/access-control/src/lib.rs` (+13 -11) 📝 `crates/auth/Cargo.toml` (+1 -1) 📝 `crates/auth/src/opaque.rs` (+6 -0) 📝 `crates/domain-model/Cargo.toml` (+2 -8) 📝 `crates/domain/src/deserialize.rs` (+1 -1) 📝 `crates/domain/src/lib.rs` (+2 -0) 📝 `crates/domain/src/public_schema.rs` (+1 -1) ➕ `crates/graphql-server/Cargo.toml` (+75 -0) ➕ `crates/graphql-server/src/api.rs` (+91 -0) 📝 `crates/graphql-server/src/lib.rs` (+0 -0) 📝 `crates/graphql-server/src/mutation.rs` (+16 -14) 📝 `crates/graphql-server/src/query.rs` (+15 -16) ➕ `crates/ldap/Cargo.toml` (+56 -0) ➕ `crates/ldap/src/compare.rs` (+240 -0) 📝 `crates/ldap/src/core/error.rs` (+0 -0) 📝 `crates/ldap/src/core/group.rs` (+12 -15) 📝 `crates/ldap/src/core/mod.rs` (+0 -0) 📝 `crates/ldap/src/core/user.rs` (+13 -14) _...and 43 more files_ </details> ### 📄 Description This introduces many small crates, clarifying the dependencies and cleaning up the code. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>