💬 l0rinc commented on pull request "p2p: add more bad ports":
(https://github.com/bitcoin/bitcoin/pull/32826#discussion_r2173857150)
👍 https://www.postgresql.org/docs/current/app-postgres.html

📝 hebasto opened a pull request: "doc: Update NetBSD Build Guide"
(https://github.com/bitcoin/bitcoin/pull/32828)
The `boost-headers` package does not provide CMake configuration files required since https://github.com/bitcoin/bitcoin/pull/32667. Install the `boost` package instead.

📝 hebasto converted_to_draft a pull request: "doc: Update NetBSD Build Guide"
(https://github.com/bitcoin/bitcoin/pull/32828)
The `boost-headers` package does not provide CMake configuration files required since https://github.com/bitcoin/bitcoin/pull/32667. Install the `boost` package instead.

💬 jlopp commented on pull request "p2p: add more bad ports":
(https://github.com/bitcoin/bitcoin/pull/32826#discussion_r2173890219)
implemented

📝 kevkevinpal opened a pull request: "threading: use correct mutex name in reverse_lock fatal error messages"
(https://github.com/bitcoin/bitcoin/pull/32829)
"Now that REVERSE_LOCK requires the name of the actual mutex, it can be used for better error messages." - theuni

This is a follow-up to this comment https://github.com/bitcoin/bitcoin/pull/32465#issuecomment-2981287545

I just cherry-picked the commit 85c2848eb575f4abaa81fdd4e8f3b2048693dd98

🤔 l0rinc reviewed a pull request: "p2p: add more bad ports"
(https://github.com/bitcoin/bitcoin/pull/32826#pullrequestreview-2969413138)
ACK 8716fd2f4a00b07426e54a1491af421d6345f708

💬 l0rinc commented on pull request "p2p: add more bad ports":
(https://github.com/bitcoin/bitcoin/pull/32826#discussion_r2173890816)
might as well inline it now that it's only used once
```suggestion
BOOST_CHECK_EQUAL(std::ranges::count_if(ports, IsBadPort), 85);
```

💬 l0rinc commented on pull request "p2p: add more bad ports":
(https://github.com/bitcoin/bitcoin/pull/32826#discussion_r2173892111)
👍, looks like this contains the end as well, no +1 needed.
Checked with `BOOST_CHECK_EQUAL(std::ranges::count_if(ports, [&](const int i) { return i == std::numeric_limits<uint16_t>::max(); }), 1);` to make sure.

---

Note though that this is a substantial memory allocation, the previous suggestion was lazy.
For the record here's the complete lazy solution I suggested (with fixed imports):
```patch
diff --git a/src/test/netbase_tests.cpp b/src/test/netbase_tests.cpp
index e3bf482873..
...

💬 kevkevinpal commented on pull request "p2p: add more bad ports":
(https://github.com/bitcoin/bitcoin/pull/32826#issuecomment-3017035899)
ACK [8716fd2](https://github.com/bitcoin/bitcoin/pull/32826/commits/8716fd2f4a00b07426e54a1491af421d6345f708)

It makes sense to me to add these ports since they're popular and used. I double-checked the links @l0rinc provided and also did a quick search myself to make sure they were correct

💬 jlopp commented on pull request "p2p: add more bad ports":
(https://github.com/bitcoin/bitcoin/pull/32826#discussion_r2173897398)
done

👋 hebasto's pull request is ready for review: "build, docs: Fix Boost-related issues on NetBSD"
(https://github.com/bitcoin/bitcoin/pull/32828)

💬 ariard commented on pull request "BIP-119 (OP_CHECKTEMPLATEVERIFY) (regtest only)":
(https://github.com/bitcoin/bitcoin/pull/31989#issuecomment-3017163445)
Opened a pull request on Jamesob repository with the suggested changes: due to current block sigs exhaustions exposure:
https://github.com/jamesob/bitcoin/pull/3

Changes motivated by current exposure to block signature overflow attacks and it’s quite simple to fix.

⚠️ Raimo33 opened an issue: "Redundant else statements in SRC/logging.cpp"
(https://github.com/bitcoin/bitcoin/issues/32830)
static std::optional<BCLog::Level> GetLogLevel(std::string_view level_str)
{
if (level_str == "trace") {
return BCLog::Level::Trace;
} else if (level_str == "debug") {
return BCLog::Level::Debug;
} else if (level_str == "info") {
return BCLog::Level::Info;
} else if (level_str == "warning") {
return BCLog::Level::Warning;
} else if (level_str == "error") {
return BCLog::Level::Error;
} else {
return std::nullopt;
}
}

🤔 mzumsande reviewed a pull request: "mempool: Avoid needless vtx iteration in `removeForBlock` during IBD"
(https://github.com/bitcoin/bitcoin/pull/32827#pullrequestreview-2969577114)
"known to be empty" / "always empty" is too strong:
It's quite common for nodes that are not online 24/7 to be in IBD (because they have to catch up a few weeks/months) but have a non-empty mempool. This doesn't affect the approach, just the PR / commit message description.

⚠️ jasminemoore0410 opened an issue: "FUNDS RECLAIMER COMPANY / BEST RECOVERY SERVICE TO HELP YOU GET BACK YOUR LOST CRYPTO"
(https://github.com/bitcoin/bitcoin/issues/32831)
### Motivation

CONTACT INFO BELOW

WhatsApp:: + 1 3 6 1 2 5 0 4 1 1 0

Email:: fundsreclaimer@consultant.com

Telegram:: @fundsreclaimercompany0

A new breed of scam has emerged in the USA, and unfortunately, I became one of its victims. I was drawn in by the allure of an online gambling platform that promised easy winnings and thrilling experiences. The site looked legitimate, and I was excited to deposit my cryptocurrency, specifically 4.5 BTC, believing I was making a smart investment. Howev
...

💬 l0rinc commented on pull request "mempool: Avoid needless vtx iteration in `removeForBlock` during IBD":
(https://github.com/bitcoin/bitcoin/pull/32827#issuecomment-3017884339)
Thanks, updated - though I don't think that's "Initial", just regular Block Download.

💬 l0rinc commented on issue "Redundant else statements in SRC/logging.cpp":
(https://github.com/bitcoin/bitcoin/issues/32830#issuecomment-3017886926)
Please stop spamming, this isn't useful

✅ maflcko closed an issue: "Redundant else statements in SRC/logging.cpp"
(https://github.com/bitcoin/bitcoin/issues/32830)

💬 maflcko commented on pull request "p2p: add more bad ports":
(https://github.com/bitcoin/bitcoin/pull/32826#issuecomment-3017950506)
Please squash your commits according to https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#squashing-commits

💬 maflcko commented on pull request "docs: add guidance on initialism capitalisation in PascalCase identifiers.":
(https://github.com/bitcoin/bitcoin/pull/32720#issuecomment-3017954531)
Please squash your commits according to https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#squashing-commits