[PR #1096] [MERGED] Cleanup log messages for ignored attribute warnings #1115

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

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/1096
Author: @maystar
Created: 2/3/2025
Status: Merged
Merged: 2/3/2025
Merged by: @nitnelave

Base: mainHead: cleanup-logs-for-attribute-warnings


📝 Commits (3)

  • f992bea Fix warn log message for unrecognized user attribute
  • 59a76ec Merge pull request #1 from maystar/fix-log-message-user-attribute
  • 67482ee Reduce log messages by remove line break and remove visible \n\ sequence

📊 Changes

2 files changed (+2 additions, -4 deletions)

View changed files

📝 server/src/domain/ldap/group.rs (+1 -2)
📝 server/src/domain/ldap/user.rs (+1 -2)

📄 Description

When debugging lldap logs I recognized that the output looks strange using syslog and grafana.
There are two outputs for each warnings, the first one contains the visible \n\ sequence.
Additional the user message contains the string group attribute .

Be aware: I've no clue about rust development!


🔄 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/1096 **Author:** [@maystar](https://github.com/maystar) **Created:** 2/3/2025 **Status:** ✅ Merged **Merged:** 2/3/2025 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `cleanup-logs-for-attribute-warnings` --- ### 📝 Commits (3) - [`f992bea`](https://github.com/lldap/lldap/commit/f992bea0f08bf86ab2472ef894465791e03cd9fc) Fix warn log message for unrecognized user attribute - [`59a76ec`](https://github.com/lldap/lldap/commit/59a76eca1dd79d34783ce84d1cbe1a2efc608ab6) Merge pull request #1 from maystar/fix-log-message-user-attribute - [`67482ee`](https://github.com/lldap/lldap/commit/67482ee557557aab25b58c57b1a4d9b9bc4fff60) Reduce log messages by remove line break and remove visible \n\ sequence ### 📊 Changes **2 files changed** (+2 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `server/src/domain/ldap/group.rs` (+1 -2) 📝 `server/src/domain/ldap/user.rs` (+1 -2) </details> ### 📄 Description When debugging lldap logs I recognized that the output looks strange using syslog and grafana. There are two outputs for each warnings, the first one contains the visible `\n\` sequence. Additional the user message contains the string ` group attribute` . Be aware: I've no clue about rust development! --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 09:10:58 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/lldap-lldap#1115
No description provided.