[PR #338] [MERGED] server: refactor sql backend handler #688

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/338
Author: @nitnelave
Created: 10/18/2022
Status: ✅ Merged
Merged: 10/18/2022
Merged by: @nitnelave

Base: main ← Head: refactoring

📝 Commits (1)

100b50f server: refactor sql backend handler

📊 Changes

9 files changed (+1176 additions, -891 deletions)

View changed files

📝 server/src/domain/handler.rs (+24 -12)
📝 server/src/domain/mod.rs (+2 -0)
📝 server/src/domain/sql_backend_handler.rs (+37 -814)
➕ server/src/domain/sql_group_backend_handler.rs (+301 -0)
📝 server/src/domain/sql_opaque_handler.rs (+51 -40)
➕ server/src/domain/sql_user_backend_handler.rs (+737 -0)
📝 server/src/infra/ldap_handler.rs (+12 -7)
📝 server/src/infra/tcp_backend_handler.rs (+11 -17)
📝 server/src/main.rs (+1 -1)

📄 Description

And add some missing tests

_{🔄 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/338 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 10/18/2022 **Status:** ✅ Merged **Merged:** 10/18/2022 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `refactoring` --- ### 📝 Commits (1) - [`100b50f`](https://github.com/lldap/lldap/commit/100b50fe37560bdd5efa086c4f102642d123b677) server: refactor sql backend handler ### 📊 Changes **9 files changed** (+1176 additions, -891 deletions) <details> <summary>View changed files</summary> 📝 `server/src/domain/handler.rs` (+24 -12) 📝 `server/src/domain/mod.rs` (+2 -0) 📝 `server/src/domain/sql_backend_handler.rs` (+37 -814) ➕ `server/src/domain/sql_group_backend_handler.rs` (+301 -0) 📝 `server/src/domain/sql_opaque_handler.rs` (+51 -40) ➕ `server/src/domain/sql_user_backend_handler.rs` (+737 -0) 📝 `server/src/infra/ldap_handler.rs` (+12 -7) 📝 `server/src/infra/tcp_backend_handler.rs` (+11 -17) 📝 `server/src/main.rs` (+1 -1) </details> ### 📄 Description And add some missing tests --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>