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
  1. ViniciusCestarii commented at 4:17 pm on March 16, 2026: contributor

    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.

  2. log: add socket connected info message and demote destroy logs to debug 98f1352159
  3. DrahtBot commented at 4:18 pm on March 16, 2026: none

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK ryanofsky

    If your review is incorrectly listed, please copy-paste <!–meta-tag:bot-skip–> into the comment that the bot should ignore.

    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.

  4. ryanofsky approved
  5. ryanofsky commented at 6:36 pm on March 23, 2026: collaborator
    Code review ACK 98f1352159de43355499f16879d39e86b7990e35. Thanks for this the fix, these changes make sense.
  6. ryanofsky merged this on Mar 23, 2026
  7. ryanofsky closed this on Mar 23, 2026


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/libmultiprocess. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-03-29 21:30 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me