💬 jonatack commented on pull request "kernel: Remove UniValue from kernel library":
(https://github.com/bitcoin/bitcoin/pull/28113#discussion_r1271540053)
https://github.com/bitcoin/bitcoin/commit/29e31d894bd8bedb55c3a5a2d4d2681ba10a9b88 (feel free to ignore) I wouldn't find review harder if the missing brackets were added

```cpp
if (v.isStr()) {
strHex = v.getValStr();
}
if (!IsHex(strHex)) {
throw std::runtime_error(strName + " must be hexadecimal string (not '" + strHex + "')");
}
```

💬 jonatack commented on pull request "refactor: Add util::Result failure values, multiple error and warning messages":
(https://github.com/bitcoin/bitcoin/pull/25665#discussion_r1271547869)
> I should probably add a tutorial-style document that describes how to use the Result class with standalone functions that return util::Result, chained functions that return util::Result values of the same type, and chained functions that return util::Result values of different types.

I think this would be valuable, either in `util/result.h` directly or in `doc/developer-notes.md`.

⚠️ jonatack opened an issue: "p2p_getaddr_caching.py failure in TSan CI"
(https://github.com/bitcoin/bitcoin/issues/28133)
### Is there an existing issue for this?

- [X] I have searched the existing issues

### Current behaviour

https://cirrus-ci.com/task/5471865133596672?logs=ci#L3981


### Expected behaviour

https://cirrus-ci.com/task/5471865133596672?logs=ci#L3981


### Steps to reproduce

https://cirrus-ci.com/task/5471865133596672?logs=ci#L3981


### Relevant log output

https://cirrus-ci.com/task/5471865133596672?logs=ci#L3981

```
test 2023-07-22T19:16:17.696000Z TestFramework (ERROR): Assertion fa
...

💬 emc99 commented on pull request "Enable full-rbf by default":
(https://github.com/bitcoin/bitcoin/pull/28132#issuecomment-1646954964)
When you say "by default", do you mean that full-rbf would come by default as part of IBD or when you update Bitcoin Core? When would full-rbf be "by default"?

💬 hebasto commented on issue "ci: Future of macOS and Windows MSVC CI tasks":
(https://github.com/bitcoin/bitcoin/issues/28098#issuecomment-1646955146)
Among free options, CircleCI looks better than others, IMO:
- https://circleci.com/docs/plan-free/
- https://circleci.com/pricing/

> Personally, I don't use macOS nor Windows, and I can't recommend it to anyone, so I don't mind if the tasks are removed.

We are responsible for quality of every release we are [shipping](https://bitcoincore.org/en/download/). And the user is free to make their own choice.

💬 emc99 commented on pull request "fuzz: Re-enable symbolize=1 in ASAN_OPTIONS":
(https://github.com/bitcoin/bitcoin/pull/28124#issuecomment-1646955442)
How do you fuzz? What does it mean to 'fuzz'?

💬 emc99 commented on pull request "build: pass sanitize flags to instrument libsecp256k1 code":
(https://github.com/bitcoin/bitcoin/pull/27991#issuecomment-1646956148)
> Maybe mark as draft for as long as CI is red?

When does CI happen? Why would CI be red in this case?

📝 jonatack opened a pull request: "rpc, util: deduplicate AmountFromValue() using util::Result"
(https://github.com/bitcoin/bitcoin/pull/28134)
See commit messages for details.

💬 TheCharlatan commented on pull request "kernel: Remove UniValue from kernel library":
(https://github.com/bitcoin/bitcoin/pull/28113#issuecomment-1646962287)
Updated b89567f51ade926af8c918e4787046b7ccec8eb0 -> a3774d1b2a5ce9aa6d6d3cedc2c9b9a5d2f68240 ([kernelRmUnivalue_4](https://github.com/TheCharlatan/bitcoin/tree/kernelRmUnivalue_4) -> [kernelRmUnivalue_5](https://github.com/TheCharlatan/bitcoin/tree/kernelRmUnivalue_5), [compare](https://github.com/TheCharlatan/bitcoin/compare/kernelRmUnivalue_4..kernelRmUnivalue_5))

* Addressed @jonatack's [comment](https://github.com/bitcoin/bitcoin/pull/28113#discussion_r1271533741), fixing IWYU.
* Addres
...

💬 jonatack commented on pull request "kernel: Remove UniValue from kernel library":
(https://github.com/bitcoin/bitcoin/pull/28113#issuecomment-1646964712)
ACK a3774d1b2a5ce9aa6d6d3cedc2c9b9a5d2f68240

💬 petertodd commented on pull request "Enable full-rbf by default":
(https://github.com/bitcoin/bitcoin/pull/28132#issuecomment-1646966801)
> When you say "by default", do you mean that full-rbf would come by default as part of IBD or when you update Bitcoin Core? When would full-rbf be "by default"?

This pull-req has nothing to do with Initial Block Download (IBD).

It simply changes the default for the `-mempoolfullrbf` option to true/enabled. Previously the default was false/disabled. Users who update Bitcoin Core to a version containing this change would by default propagate and mine full-rbf replacements unless they had ch
...

💬 theStack commented on pull request "BIP324 ciphersuite":
(https://github.com/bitcoin/bitcoin/pull/28008#discussion_r1271567848)
`salt` is currently not used anywhere.

👋 jonatack's pull request is ready for review: "rpc, util: deduplicate AmountFromValue() using util::Result"
(https://github.com/bitcoin/bitcoin/pull/28134)

📝 bitcoinfinancier opened a pull request: "v 25.0.3"
(https://github.com/bitcoin/bitcoin/pull/28135)
semgrep ci integration for security and optimized images with AI to reduce storage and enhance quality.

💬 jonatack commented on pull request "Remove C-style const-violating cast, Use reinterpret_cast":
(https://github.com/bitcoin/bitcoin/pull/28127#issuecomment-1646980929)
Concept ACK

💬 Ayms commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1646981789)
@luke-jr the idea is not to store a video on bitcoin but indeed a reference to it like bittorrent
Now if someone wants to store things in a full block nobody can avoid this
Then this change just make things easier to avoid unwanted practices

📝 bitcoinfinancier converted_to_draft a pull request: "v 25.0.3"
(https://github.com/bitcoin/bitcoin/pull/28135)
semgrep ci integration for security and optimized images with AI to reduce storage and enhance quality.

👋 bitcoinfinancier's pull request is ready for review: "v 25.0.3"
(https://github.com/bitcoin/bitcoin/pull/28135)

📝 bitcoinfinancier converted_to_draft a pull request: "v 25.0.3"
(https://github.com/bitcoin/bitcoin/pull/28135)
semgrep ci integration for security and optimized images with AI to reduce storage and enhance quality.

💬 russeree commented on issue "Bitcoin Core v25.0 Crashes":
(https://github.com/bitcoin/bitcoin/issues/28119#issuecomment-1646990896)
Corrupted timechain or hardware issue.

```
2023-07-23T14:28:38Z ERROR: ReadBlockFromDisk: Deserialize or I/O error - AutoFile::read: fread failed: iostream error at FlatFilePos(nFile=9, nPos=53762556)
2023-07-23T14:28:38Z *** Failed to read block
2023-07-23T14:28:38Z Error: A fatal internal error occurred, see debug.log for details
```

Rhe broken file is nfile = is the blk0009.dat. This could be a software or hardware issue.

To debug and fix this issue there are few possible solut
...