net: guard RemoveLocal() log with fLogIPs #34665

pull chriszeng1010 wants to merge 1 commits into bitcoin:master from chriszeng1010:fix-removelocal-logips changing 1 files +3 −1
  1. chriszeng1010 commented at 7:40 pm on February 24, 2026: none

    RemoveLocal() logs the local IP address unconditionally via LogInfo(). Other logging in net.cpp respects the fLogIPs flag to avoid leaking IP addresses when the user has not opted in with -logips.

    Gate the log behind fLogIPs for consistency.

    Relates to #34458.

  2. net: guard RemoveLocal() log with fLogIPs d2a86c3ba9
  3. DrahtBot added the label P2P on Feb 24, 2026
  4. DrahtBot commented at 7:40 pm on February 24, 2026: contributor

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

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  5. sedited commented at 7:33 am on February 25, 2026: contributor
    This is already covered by #34458. Lets focus review over there.
  6. sedited closed this on Feb 25, 2026


github-metadata-mirror

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

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