[PR #336] [MERGED] Refactor ldap_handler #687

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

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

Base: main ← Head: refactoring

📝 Commits (2)

85b4fc4 server: refactor ldap_handler
3c8f54d nextest: add configuration

📊 Changes

8 files changed (+822 additions, -787 deletions)

View changed files

➕ .config/nextest.toml (+2 -0)
➕ server/src/domain/ldap/error.rs (+17 -0)
➕ server/src/domain/ldap/group.rs (+218 -0)
➕ server/src/domain/ldap/mod.rs (+4 -0)
➕ server/src/domain/ldap/user.rs (+236 -0)
➕ server/src/domain/ldap/utils.rs (+157 -0)
📝 server/src/domain/mod.rs (+1 -0)
📝 server/src/infra/ldap_handler.rs (+187 -787)

📄 Description

Split it into several files, move them into the domain folder, introduce
LdapError for better control flow.

_{🔄 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/336 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 10/17/2022 **Status:** ✅ Merged **Merged:** 10/17/2022 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `refactoring` --- ### 📝 Commits (2) - [`85b4fc4`](https://github.com/lldap/lldap/commit/85b4fc44066cc8012389eb53a42db4c15122a177) server: refactor ldap_handler - [`3c8f54d`](https://github.com/lldap/lldap/commit/3c8f54dd5ae2674719859ac7d8a0f2e90e126e2a) nextest: add configuration ### 📊 Changes **8 files changed** (+822 additions, -787 deletions) <details> <summary>View changed files</summary> ➕ `.config/nextest.toml` (+2 -0) ➕ `server/src/domain/ldap/error.rs` (+17 -0) ➕ `server/src/domain/ldap/group.rs` (+218 -0) ➕ `server/src/domain/ldap/mod.rs` (+4 -0) ➕ `server/src/domain/ldap/user.rs` (+236 -0) ➕ `server/src/domain/ldap/utils.rs` (+157 -0) 📝 `server/src/domain/mod.rs` (+1 -0) 📝 `server/src/infra/ldap_handler.rs` (+187 -787) </details> ### 📄 Description Split it into several files, move them into the domain folder, introduce `LdapError` for better control flow. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>