Bugfix: Wallet: Skip inaccessible directories rather than abort the wallet list entirely #32812

pull luke-jr wants to merge 1 commits into bitcoin:master from luke-jr:fix_wallet_list_ignore_eperm changing 1 files +1 −1
  1. luke-jr commented at 3:05 pm on June 25, 2025: member
    Followup to #32736 to restore the intended behaviour (absent the detailed logging which appears to not be practical with std::filesystem).
  2. Bugfix: Wallet: Skip inaccessible directories rather than abort the wallet list entirely 2be6089394
  3. DrahtBot commented at 3:05 pm on June 25, 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/32812.

    Reviews

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

  4. fanquake requested review from rkrux on Jun 25, 2025
  5. fanquake requested review from hodlinator on Jun 25, 2025
  6. maflcko commented at 3:25 pm on June 25, 2025: member
    pretty sure the tests are going to fail now? maybe the ci should go back to run in a user account, if possible?
  7. luke-jr commented at 6:51 pm on June 25, 2025: member
    This actually isn’t necessary. Unreadable subdirectories get skipped explicitly when checking for their wallet.dat fails.
  8. luke-jr closed this on Jun 25, 2025


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-07 21:13 UTC

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