💬 Retropex commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666492502)
@zkfrio Do you think that giving the choice of the mempool policy is attacking the network?

💬 Retropex commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666494844)
> The vast majority of the recent UTXO set size increase (which mostly happened in the last four months, not seven) comes not from [bare multisig](https://transactionfee.info/charts/inputs-and-outputs-p2ms/) but from [BRC-20](https://transactionfee.info/charts/inputs-and-outputs-p2tr/). Bare multisig peaked in April, barely crossing 10k unspent outputs a day, and currently adds only a few hundred daily, while BRC-20 still keeps adding over 50k unspent outputs every day.

I know it, unfortunate
...

💬 zkfrio commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666496157)
> @zkfrio Do you think that giving the choice of the mempool policy is attacking the network?

I was responding to 'ordislow' related comment. It's an organized attack that would not improve anything.

💬 dergoegge commented on pull request "Use shared_ptr for CNode inside CConnman":
(https://github.com/bitcoin/bitcoin/pull/28222#issuecomment-1666508138)
Also see #10738

💬 petertodd commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666512036)
@HenrikJannsen

> Ideas like https://twitter.com/hashtag/ordislow with delaying blocks containing ordinals might be worth considering in more detail. I don't understand why Bitcoin devs are not more engaged in those discussions.

Bitcoin devs haven't engaged with those discussions because #ordislow is a dumb idea that is very unlikely to work; to the extent it does work, it will simply act to further centralize mining. #ordislow is fundamentally an attempt at censorship of blocks (informati
...

💬 petertodd commented on pull request "policy: Enable full-rbf by default":
(https://github.com/bitcoin/bitcoin/pull/28132#issuecomment-1666513527)
> why not focus efforts on reducing the need for Replace by Fee to begin with?

As I and others have explained above in the linked resources, eg https://petertodd.org/2023/fullrbf-multiparty-protocols and https://petertodd.org/2023/why-you-should-run-mempoolfullrbf, it is impossible to eliminate the need for transaction replacement.

💬 willcl-ark commented on pull request "Use shared_ptr for CNode inside CConnman":
(https://github.com/bitcoin/bitcoin/pull/28222#issuecomment-1666517688)
> Also see #10738

Thanks, I hadn't seen this one before. Will take a look tonight

💬 luke-jr commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666522472)
>Allows for including signature by a third party other than that signing the keys controlling the inputs. E.g., service provider.

🚩🚩🚩 Sounds like maybe this is tied to some kind of KYC nonsense?

There shouldn't be a "service provider" to begin with. (And if you really need such a signature, you could put it INSIDE the data which is being hashed.)

>People were putting garbage and "spamming" the blockchain in the past, spamming is rampant currently and I have never ever seen a convinci
...

💬 luke-jr commented on pull request "Bech32 LocateErrors "Bech32 string too short" case":
(https://github.com/bitcoin/bitcoin/pull/28160#issuecomment-1666526030)
The new error_locations looks worse/broken...

💬 luke-jr commented on pull request "refactor: Revert additional univalue check in ParseSighashString":
(https://github.com/bitcoin/bitcoin/pull/28162#discussion_r1285070240)
Might also want to catch it NOT throwing an error when it ought to.

💬 luke-jr commented on pull request "script: throw disabled err for op_ver and its variants":
(https://github.com/bitcoin/bitcoin/pull/28169#discussion_r1285070870)
How does OP_VER differ from a non-existent opcode?

💬 luke-jr commented on pull request "ParseHDKeypath: support h as hardened marker":
(https://github.com/bitcoin/bitcoin/pull/28192#issuecomment-1666530013)
Seems likely someone somewhere will want to combine two derivation paths, which might use different markers.

💬 achow101 commented on pull request "script: throw disabled err for op_ver and its variants":
(https://github.com/bitcoin/bitcoin/pull/28169#discussion_r1285078233)
It's actually defined and exists in documentation.

💬 luke-jr commented on pull request "mempool: Persist with XOR":
(https://github.com/bitcoin/bitcoin/pull/28207#issuecomment-1666541683)
Concept ACK, I don't think we need to concern with external programs reading this. If they do exist, they can adapt easily.

💬 luke-jr commented on pull request "mempool: Persist with XOR":
(https://github.com/bitcoin/bitcoin/pull/28207#issuecomment-1666541746)
(Although downgrading might be something we want to support?)

💬 luke-jr commented on pull request "Bump python minimum version to 3.9":
(https://github.com/bitcoin/bitcoin/pull/28211#issuecomment-1666542228)
>CentOS Stream also ships with 3.9

Isn't Stream a rolling release? What about RHEL?

💬 BBezaire commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666543561)
Concept NACK

My node isn't your personal hard drive. It's for financial transactions, period.

We're already forced to store ordinals for free, while miners profit greatly off priority fees. Such a change will only make things worse for node operators, and destroy Bitcoin's credibility as a crypto currency.

This is a slap in the face to everything Satoshi worked towards.

💬 luke-jr commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#discussion_r1285087710)
??? this change makes no sense to me. `i` is an `int`, but `IsStandardTx` returns a bool...

💬 luke-jr commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#discussion_r1285087863)
See b90ef78f7ab564fb5948583e7a5b5399f9798301 for how I fixed this in Knots

💬 jonatack commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#discussion_r1285088166)
@luke-jr an `int` of 0 is falsey, non-zero is truthy.