[PR #667] [MERGED] Update tracing_forest and ldap3_proto #881

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/667
Author: @nitnelave
Created: 9/10/2023
Status: ✅ Merged
Merged: 9/10/2023
Merged by: @nitnelave

Base: main ← Head: updates

📝 Commits (5)

190847d cargo: Fix getrandom dependency
5b4cd80 server: Update tracing-forest and take advantage of the span fields
f31500d server: Upgrade ldap3_proto
7c3e6c6 server: Switch tests to pretty_assertions
d7c91ee server: fix clippy warnings

📊 Changes

25 files changed (+108 additions, -99 deletions)

View changed files

📝 Cargo.lock (+36 -33)
📝 Cargo.toml (+2 -0)
📝 auth/Cargo.toml (+1 -1)
📝 migration-tool/src/ldap.rs (+2 -2)
📝 server/Cargo.toml (+3 -2)
📝 server/src/domain/handler.rs (+2 -1)
📝 server/src/domain/ldap/group.rs (+1 -2)
📝 server/src/domain/ldap/user.rs (+1 -2)
📝 server/src/domain/ldap/utils.rs (+2 -2)
📝 server/src/domain/sql_backend_handler.rs (+1 -0)
📝 server/src/domain/sql_group_backend_handler.rs (+7 -11)
📝 server/src/domain/sql_opaque_handler.rs (+3 -3)
📝 server/src/domain/sql_schema_backend_handler.rs (+1 -0)
📝 server/src/domain/sql_tables.rs (+1 -0)
📝 server/src/domain/sql_user_backend_handler.rs (+10 -17)
📝 server/src/domain/types.rs (+15 -1)
📝 server/src/infra/auth_service.rs (+1 -2)
📝 server/src/infra/configuration.rs (+1 -0)
📝 server/src/infra/db_cleaner.rs (+0 -2)
📝 server/src/infra/graphql/query.rs (+1 -0)

...and 5 more files

📄 Description

This should solve #663

_{🔄 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/667 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 9/10/2023 **Status:** ✅ Merged **Merged:** 9/10/2023 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `updates` --- ### 📝 Commits (5) - [`190847d`](https://github.com/lldap/lldap/commit/190847dd4136f5d0f350ff3adf8744f92bbc2f63) cargo: Fix getrandom dependency - [`5b4cd80`](https://github.com/lldap/lldap/commit/5b4cd80f678f64a651b6fe176441f6e1f1f636a0) server: Update tracing-forest and take advantage of the span fields - [`f31500d`](https://github.com/lldap/lldap/commit/f31500dd9595278dc650ff52cf79e9cb6af07be8) server: Upgrade ldap3_proto - [`7c3e6c6`](https://github.com/lldap/lldap/commit/7c3e6c6c1488517453d7cf256eb650ab6a50b2f0) server: Switch tests to pretty_assertions - [`d7c91ee`](https://github.com/lldap/lldap/commit/d7c91eedd91d05f2f9155f2b35f93f6bdac4c0fe) server: fix clippy warnings ### 📊 Changes **25 files changed** (+108 additions, -99 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+36 -33) 📝 `Cargo.toml` (+2 -0) 📝 `auth/Cargo.toml` (+1 -1) 📝 `migration-tool/src/ldap.rs` (+2 -2) 📝 `server/Cargo.toml` (+3 -2) 📝 `server/src/domain/handler.rs` (+2 -1) 📝 `server/src/domain/ldap/group.rs` (+1 -2) 📝 `server/src/domain/ldap/user.rs` (+1 -2) 📝 `server/src/domain/ldap/utils.rs` (+2 -2) 📝 `server/src/domain/sql_backend_handler.rs` (+1 -0) 📝 `server/src/domain/sql_group_backend_handler.rs` (+7 -11) 📝 `server/src/domain/sql_opaque_handler.rs` (+3 -3) 📝 `server/src/domain/sql_schema_backend_handler.rs` (+1 -0) 📝 `server/src/domain/sql_tables.rs` (+1 -0) 📝 `server/src/domain/sql_user_backend_handler.rs` (+10 -17) 📝 `server/src/domain/types.rs` (+15 -1) 📝 `server/src/infra/auth_service.rs` (+1 -2) 📝 `server/src/infra/configuration.rs` (+1 -0) 📝 `server/src/infra/db_cleaner.rs` (+0 -2) 📝 `server/src/infra/graphql/query.rs` (+1 -0) _...and 5 more files_ </details> ### 📄 Description This should solve #663 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>