Skip to content

Conversation

@nicozanf
Copy link
Collaborator

(AI generated)

This PR ensures that the logger's level is set to the lowest handler level specified in the LOGGERS configuration. Previously, even if a handler was set to DEBUG, the logger itself could filter out lower-level messages (like DEBUG), resulting in missing log output. By explicitly setting root.setLevel(min_level), all messages at or above the configured level are processed and passed to the handlers as expected.

nicozanf added 2 commits June 28, 2025 11:41
(AI generated)
**Suggested PR Comment:**

This PR ensures that the logger's level is set to the lowest handler level specified in the `LOGGERS` configuration.  
Previously, even if a handler was set to `DEBUG`, the logger itself could filter out lower-level messages (like `DEBUG`), resulting in missing log output.  
By explicitly setting `root.setLevel(min_level)`, all messages at or above the configured level are processed and passed to the handlers as expected.
@nicozanf
Copy link
Collaborator Author

It closes #967

@nicozanf nicozanf linked an issue Jun 29, 2025 that may be closed by this pull request
@mdipierro mdipierro merged commit c5e3a24 into master Jun 29, 2025
5 checks passed
@nicozanf nicozanf deleted the nicozanf-patch-3 branch July 20, 2025 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

debug log

3 participants