log: Properly log warnings with warn loglevel in addrdb #32933

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2507-log-warn-addrdb changing 1 files +8 −8
  1. maflcko commented at 9:27 am on July 10, 2025: member

    The logging in addrdb is confusing, because it uses LogPrintf (info level) to log warnings.

    Fix this by properly using the warn level, where needed. Also, drop unused trailing \n while touching the lines.

  2. log: Properly log warnings with warn loglevel in addrdb fa894b0f3e
  3. DrahtBot added the label Utils/log/libs on Jul 10, 2025
  4. DrahtBot commented at 9:27 am on July 10, 2025: contributor

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

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32933.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK stickies-v

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #29641 (scripted-diff: Use LogInfo over LogPrintf [WIP, NOMERGE, DRAFT] by maflcko)

    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.

  5. stickies-v approved
  6. stickies-v commented at 11:33 am on July 10, 2025: contributor
    ACK fa894b0f3e13dcc55fd42cec2c56d4aa2115194d

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: 2025-07-11 09:13 UTC

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