Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cln-plugin: Change default log level filter back to INFO #7668

Merged

Conversation

laanwj
Copy link
Contributor

@laanwj laanwj commented Sep 13, 2024

cln-plugin: Change default log level filter back to INFO

Description

In commit 60e1532 (released in crate 0.1.8), which switched the logging framework to tracing-subscriber, the default log level filter was (accidentally) set to ERROR and above, instead of INFO and above.

Change this back to INFO as it was before. It can still be overridden with environment variable CLN_PLUGIN_LOG.

Follows the example in https://docs.rs/tracing-subscriber/latest/tracing_subscriber/filter/struct.EnvFilter.html#method.from_env

Related Issues

Changes Made

Changelog-Fixed: cln-plugin: Change default log level filter back to INFO

Checklist

Ensure the following tasks are completed before submitting the PR:

  • Changelog has been added in relevant commit/s.
  • N/A Tests have been added or updated to cover the changes.
  • N/A Documentation has been updated as needed.
  • N/A Any relevant comments or TODOs have been addressed or removed.

In commit 60e1532 (released in crate 0.1.8), which switched the
logging framework to tracing-subscriber, the default log level filter
was (accidentally) set to ERROR and above, instead of INFO and above.

Change this back to INFO as it was before. It can still be overridden
with CLN_PLUGIN_LOG.

Follows the example in https://docs.rs/tracing-subscriber/latest/tracing_subscriber/filter/struct.EnvFilter.html#method.from_env

Closes ElementsProject#7658.

Changelog-Fixed: cln-plugin: Change default log level filter back to INFO
Copy link
Collaborator

@vincenzopalazzo vincenzopalazzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 9d96c8c

@rustyrussell rustyrussell added this to the v24.08.1 milestone Sep 16, 2024
@rustyrussell rustyrussell merged commit 9e8dd15 into ElementsProject:master Sep 17, 2024
38 checks passed
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.

rust cln-plugin log messages dropped if CLN_PLUGIN_LOG isn't set
3 participants