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

feat: change log level ffi comms #4973

Merged

Conversation

SWvheerden
Copy link
Collaborator

Description

Reduces the log level of comms logs to be Warn or higher, unlike before it was Debug or higher

How Has This Been Tested?

manual

@stringhandler stringhandler changed the title feat: change log leve ffi comms feat: change log level ffi comms Nov 30, 2022
@stringhandler stringhandler merged commit 554e783 into tari-project:development Nov 30, 2022
@CjS77
Copy link
Collaborator

CjS77 commented Nov 30, 2022

Diasgree with this.
This should be set in the log config file, not hardcoded.

@stringhandler
Copy link
Collaborator

Created this issue to implement a configurable interface: #4982

@CjS77
Copy link
Collaborator

CjS77 commented Nov 30, 2022

Ah, I remember now. Yes, this is tricky in libwallet. Thanks for creating the issue.

@SWvheerden SWvheerden deleted the sw_change_log_leve_ffi branch December 7, 2022 04:39
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.

3 participants