[PR #201] [MERGED] Implement tracing logging #611

New issue

Closed

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

kerem commented

2026-02-27 08:18:11 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/201
Author: @nitnelave
Created: 6/30/2022
Status: ✅ Merged
Merged: 6/30/2022
Merged by: @nitnelave

Base: main ← Head: logging

📝 Commits (3)

539dd46 server: Add tracing logging
db424b7 server: refactor auth_service to use Results
b495d1a app: propagate change events

📊 Changes

28 files changed (+728 additions, -417 deletions)

View changed files

📝 Cargo.lock (+44 -6)
📝 app/src/components/change_password.rs (+2 -2)
📝 app/src/components/create_group.rs (+2 -2)
📝 app/src/components/create_user.rs (+2 -2)
📝 app/src/components/group_details.rs (+2 -2)
📝 app/src/components/group_table.rs (+2 -2)
📝 app/src/components/login.rs (+2 -2)
📝 app/src/components/logout.rs (+2 -2)
📝 app/src/components/remove_user_from_group.rs (+2 -2)
📝 app/src/components/reset_password_step1.rs (+2 -2)
📝 app/src/components/reset_password_step2.rs (+2 -2)
📝 app/src/components/user_details.rs (+2 -2)
📝 app/src/components/user_details_form.rs (+2 -2)
📝 app/src/components/user_table.rs (+2 -2)
📝 server/Cargo.toml (+12 -2)
📝 server/src/domain/sql_backend_handler.rs (+46 -29)
📝 server/src/domain/sql_opaque_handler.rs (+9 -12)
📝 server/src/infra/auth_service.rs (+257 -219)
📝 server/src/infra/db_cleaner.rs (+14 -15)
📝 server/src/infra/graphql/mutation.rs (+62 -2)

...and 8 more files

📄 Description

This PR instruments the code to add tracing events everywhere relevant. The events are then printed as a tree by tracing-forest, leading to a much-improved debug experience.

Fixes #17

_{🔄 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/201 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 6/30/2022 **Status:** ✅ Merged **Merged:** 6/30/2022 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `logging` --- ### 📝 Commits (3) - [`539dd46`](https://github.com/lldap/lldap/commit/539dd46ec8be8d8f458ff99678f5ee4c4c44ce7d) server: Add tracing logging - [`db424b7`](https://github.com/lldap/lldap/commit/db424b702b76dfaacc2266f6e76b34c54084a4cb) server: refactor auth_service to use Results - [`b495d1a`](https://github.com/lldap/lldap/commit/b495d1a3b46c716509be4aa183b964a429fd91f9) app: propagate change events ### 📊 Changes **28 files changed** (+728 additions, -417 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+44 -6) 📝 `app/src/components/change_password.rs` (+2 -2) 📝 `app/src/components/create_group.rs` (+2 -2) 📝 `app/src/components/create_user.rs` (+2 -2) 📝 `app/src/components/group_details.rs` (+2 -2) 📝 `app/src/components/group_table.rs` (+2 -2) 📝 `app/src/components/login.rs` (+2 -2) 📝 `app/src/components/logout.rs` (+2 -2) 📝 `app/src/components/remove_user_from_group.rs` (+2 -2) 📝 `app/src/components/reset_password_step1.rs` (+2 -2) 📝 `app/src/components/reset_password_step2.rs` (+2 -2) 📝 `app/src/components/user_details.rs` (+2 -2) 📝 `app/src/components/user_details_form.rs` (+2 -2) 📝 `app/src/components/user_table.rs` (+2 -2) 📝 `server/Cargo.toml` (+12 -2) 📝 `server/src/domain/sql_backend_handler.rs` (+46 -29) 📝 `server/src/domain/sql_opaque_handler.rs` (+9 -12) 📝 `server/src/infra/auth_service.rs` (+257 -219) 📝 `server/src/infra/db_cleaner.rs` (+14 -15) 📝 `server/src/infra/graphql/mutation.rs` (+62 -2) _...and 8 more files_ </details> ### 📄 Description This PR instruments the code to add tracing events everywhere relevant. The events are then printed as a tree by tracing-forest, leading to a much-improved debug experience. Fixes #17 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>