💬 jonatack commented on pull request "rpc: fix logging RPC when "none" values are passed, add test coverage, improve docs":
(https://github.com/bitcoin/bitcoin/pull/27231#discussion_r1131465027)
Thanks @MarcoFalke, done as follows.

```cpp
static void EnableOrDisableLogCategories(const UniValue& categories, bool enable)
{
const std::vector<UniValue>& category_values{categories.get_array().getValues()};
for (const auto& category : category_values) {
const std::string& c{category.get_str()};
if (c == "0" || c == "none") return; // no-op if one of these logging categories is passed
}
for (const auto& category : category_values) {
const std
...

💬 jonatack commented on pull request "rpc: fix logging RPC when "none" values are passed, add test coverage, improve docs":
(https://github.com/bitcoin/bitcoin/pull/27231#discussion_r1131469401)
(Noting that this logic is similar to that in `SetLoggingCategories()` in `src/init/common.cpp`).

💬 amitiuttarwar commented on pull request "rpc: Add test-only RPC getaddrmaninfo for new/tried table address count":
(https://github.com/bitcoin/bitcoin/pull/26988#issuecomment-1462609621)
light code review ACK 7c34c35b47. tested that the RPC and cli endpoints make sense & handle errors reasonably. these changes will require release notes, which can be done here or in a separate PR.

💬 amitiuttarwar commented on pull request "rpc: Add test-only RPC getaddrmaninfo for new/tried table address count":
(https://github.com/bitcoin/bitcoin/pull/26988#discussion_r1131449772)
you could add coverage for `getaddrmaninfo` with a network arg passed through

💬 willcl-ark commented on pull request "Improve address decoding errors":
(https://github.com/bitcoin/bitcoin/pull/26514#issuecomment-1462666899)
Concept ACK

💬 adamjonas commented on issue "Versionbits should have an ignore button":
(https://github.com/bitcoin/bitcoin/issues/8266#issuecomment-1462703683)
> I think this issue can be closed since the warnings were removed in https://github.com/bitcoin/bitcoin/pull/15471

Closing.

✅ adamjonas closed an issue: "Versionbits should have an ignore button"
(https://github.com/bitcoin/bitcoin/issues/8266)

💬 adamjonas commented on issue "[Linux] Add wayland support":
(https://github.com/bitcoin/bitcoin/issues/19950#issuecomment-1462705087)
Closing for reasons stated by hebasto above.

✅ adamjonas closed an issue: "[Linux] Add wayland support"
(https://github.com/bitcoin/bitcoin/issues/19950)

💬 adamjonas commented on issue "ops: Enable DNSSEC on all Bitcoin DNS Seed domain names":
(https://github.com/bitcoin/bitcoin/issues/19714#issuecomment-1462709723)
Closing since the rest of the seeders have said they are not going to implement or have not addressed for multiple years.

✅ adamjonas closed an issue: "ops: Enable DNSSEC on all Bitcoin DNS Seed domain names"
(https://github.com/bitcoin/bitcoin/issues/19714)

💬 adamjonas commented on issue "Cannot do HTTP JSON RPC request on wallet":
(https://github.com/bitcoin/bitcoin/issues/25635#issuecomment-1462721626)
Closing based on the above solution by @pinheadmz. Please ping if this doesn't resolve it.

✅ adamjonas closed an issue: "Cannot do HTTP JSON RPC request on wallet"
(https://github.com/bitcoin/bitcoin/issues/25635)

💬 adamjonas commented on issue "RPC `getblock` resulted in 500 and ReadBlockFromDisk: OpenBlockFile failed for FlatFilePos(nFile=-1, nPos=0)":
(https://github.com/bitcoin/bitcoin/issues/20978#issuecomment-1462728376)
@arcivanov any chance you'd be willing to attempt to reproduce with an updated release?

💬 adamjonas commented on issue "RPC field to indicate which of the conflicted wallet transactions is in the mempool":
(https://github.com/bitcoin/bitcoin/issues/21018#issuecomment-1462745565)
This hasn't seen any activity for a few years and doesn't seem to be making forward progress. Closing for now.

✅ adamjonas closed an issue: "RPC field to indicate which of the conflicted wallet transactions is in the mempool"
(https://github.com/bitcoin/bitcoin/issues/21018)

💬 adamjonas commented on issue "How to convert dumptxoutset rpc result to human readable JSON":
(https://github.com/bitcoin/bitcoin/issues/21670#issuecomment-1462753761)
There doesn't seem to be a lot of interest for the feature based on this issue but https://github.com/bitcoin/bitcoin/pull/24202 was pretty close.

💬 adamjonas commented on issue "Incorrect balance reported in getwalletinfo/getbalance":
(https://github.com/bitcoin/bitcoin/issues/21768#issuecomment-1462756998)
@Crypto2 can you attempt to reproduce with a updated version, please?

💬 Crypto2 commented on issue "Incorrect balance reported in getwalletinfo/getbalance":
(https://github.com/bitcoin/bitcoin/issues/21768#issuecomment-1462833684)
Yep I'll keep an eye on it if/when the network gets backed up like that again.

💬 furszy commented on pull request "Implement Mini version of BlockAssembler to calculate mining scores":
(https://github.com/bitcoin/bitcoin/pull/27021#discussion_r1131024834)
This can be a `CTransactionRef` (aka shared_ptr) and not a plain tx copy. Transaction data never changes.

Just need to adapt the constructor to call `GetSharedTx` instead.