Closes #227
Add "IPC server: socket connected." info log to match the existing "IPC server: socket connected." message, and demote clientDestroy/serverDestroy log messages from Info to Debug to reduce noise.
log: add socket connected info message and demote destroy logs to debug #258
pull ViniciusCestarii wants to merge 1 commits into bitcoin-core:master from ViniciusCestarii:log-socket-connected changing 2 files +3 −2-
ViniciusCestarii commented at 4:17 PM on March 16, 2026: contributor
-
log: add socket connected info message and demote destroy logs to debug 98f1352159
-
DrahtBot commented at 4:18 PM on March 16, 2026: none
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK ryanofsky If your review is incorrectly listed, please copy-paste <code><!--meta-tag:bot-skip--></code> into the comment that the bot should ignore.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #218 (Better error and log messages by ryanofsky)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
- ryanofsky approved
-
ryanofsky commented at 6:36 PM on March 23, 2026: collaborator
Code review ACK 98f1352159de43355499f16879d39e86b7990e35. Thanks for this the fix, these changes make sense.
- ryanofsky merged this on Mar 23, 2026
- ryanofsky closed this on Mar 23, 2026
Contributors