💬 whycorn commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666162624)
Peter's inflationista vibes in full swing, I don't want spam on my full node. Close this ASAP.
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666162624)
Peter's inflationista vibes in full swing, I don't want spam on my full node. Close this ASAP.
💬 whycorn commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666176725)
Someone exploiting covert channels to spam vs. simply welcoming all the spam. There's a difference in that.
NACK squared.
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666176725)
Someone exploiting covert channels to spam vs. simply welcoming all the spam. There's a difference in that.
NACK squared.
📝 ryanofsky opened a pull request: "assumeutxo cleanup: Move IsInitialBlockDownload & NotifyHeaderTip to ChainstateManager"
(https://github.com/bitcoin/bitcoin/pull/28218)
This change makes `IsInitialBlockDownload` and `NotifyHeaderTip` functions no longer tied to individual `Chainstate` objects. It makes them work with the `ChainstateManager` object instead so code is simpler and it is no longer possible to call them incorrectly with an inactive `Chainstate`.
This change also makes `m_cached_finished_ibd` caching easier to reason about, because now there is only one cached value instead of two (for background and snapshot chainstates) so the cached IBD state n
...
(https://github.com/bitcoin/bitcoin/pull/28218)
This change makes `IsInitialBlockDownload` and `NotifyHeaderTip` functions no longer tied to individual `Chainstate` objects. It makes them work with the `ChainstateManager` object instead so code is simpler and it is no longer possible to call them incorrectly with an inactive `Chainstate`.
This change also makes `m_cached_finished_ibd` caching easier to reason about, because now there is only one cached value instead of two (for background and snapshot chainstates) so the cached IBD state n
...
💬 cbergqvist commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666192859)
Concept NACK
@ChristopherA wrote:
> My goal is not "non-bitcoin stuff". My specific challenge is that Lightning and Nostr use public keys in a persistent way that are not rotatable, which is a bad cryptographic practice. By committing to a rotated key and some other cryptographic in advance in L1, they could be. I believe this could be as small as a signed hash and a little metadata. However, this is larger than 80-byte limit in OP_RETURN. I can now put it into an inscription-style transacti
...
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666192859)
Concept NACK
@ChristopherA wrote:
> My goal is not "non-bitcoin stuff". My specific challenge is that Lightning and Nostr use public keys in a persistent way that are not rotatable, which is a bad cryptographic practice. By committing to a rotated key and some other cryptographic in advance in L1, they could be. I believe this could be as small as a signed hash and a little metadata. However, this is larger than 80-byte limit in OP_RETURN. I can now put it into an inscription-style transacti
...
💬 zkfrio commented on pull request "set DEFAULT_PERMIT_BAREMULTISIG to false":
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666216446)
This change only affects P2P not blockchain. If bare multisig has no use case, why does it even exist in Bitcoin?
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666216446)
This change only affects P2P not blockchain. If bare multisig has no use case, why does it even exist in Bitcoin?
💬 Crazyk031 commented on pull request "set DEFAULT_PERMIT_BAREMULTISIG to false":
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666218117)
No transaction on the Bitcoin network is spam, what a bunch of censorshipnists..
This is not Bitcoin..
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666218117)
No transaction on the Bitcoin network is spam, what a bunch of censorshipnists..
This is not Bitcoin..
💬 Sun0fABeach commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666235310)
Concept ACK
Very important step to close this attack vector.
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666235310)
Concept ACK
Very important step to close this attack vector.
💬 HenrikJannsen commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666252994)
Concept NACK
I also do not see any reason why 80 bytes is not enough for the use cases which makes sense.
Making life for spammers easier is not our task. Better put effort into how to make it harder and more inconvenient for them to spam the blockchain. E.g. by making node options to block abusive behavior more popular (like using `permitbaremultisig=0` as mentioned here https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1665903498). Their business models will suffer if their suc
...
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666252994)
Concept NACK
I also do not see any reason why 80 bytes is not enough for the use cases which makes sense.
Making life for spammers easier is not our task. Better put effort into how to make it harder and more inconvenient for them to spam the blockchain. E.g. by making node options to block abusive behavior more popular (like using `permitbaremultisig=0` as mentioned here https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1665903498). Their business models will suffer if their suc
...
💬 HenrikJannsen commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666257468)
Concept ACK
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666257468)
Concept ACK
💬 furszy commented on pull request "indexes: Stop using node internal types and locking cs_main, improve sync logic":
(https://github.com/bitcoin/bitcoin/pull/24230#discussion_r1284902580)
> Yes, I am definitely looking for suggestions on how many of the first commits it would make sense to split off into a separate PR.
Ok cool. I suggested to decouple this commit only because I'm going commit by commit and it seems like an easy step forward, but will keep moving forward.
> But just to be clear, there isn't a bug here, just suboptimal behavior. If -reindex-chainstate is used and there are still block-connected notifications waiting in the queue when the index is started, it
...
(https://github.com/bitcoin/bitcoin/pull/24230#discussion_r1284902580)
> Yes, I am definitely looking for suggestions on how many of the first commits it would make sense to split off into a separate PR.
Ok cool. I suggested to decouple this commit only because I'm going commit by commit and it seems like an easy step forward, but will keep moving forward.
> But just to be clear, there isn't a bug here, just suboptimal behavior. If -reindex-chainstate is used and there are still block-connected notifications waiting in the queue when the index is started, it
...
💬 hsjoberg commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666268867)
Concept ACK.
It's absurd to keep such strict size limitations when ordinal inscriptions are abusing witness space and witness discount anyway.
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666268867)
Concept ACK.
It's absurd to keep such strict size limitations when ordinal inscriptions are abusing witness space and witness discount anyway.
💬 linkinparkrulz commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666294718)
>
Best practices in accordance with who?
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666294718)
>
Best practices in accordance with who?
💬 linkinparkrulz commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666295024)
"It would also promote the adoption of best practices."
Best practices in accordance with who?
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666295024)
"It would also promote the adoption of best practices."
Best practices in accordance with who?
💬 vostrnad commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666333797)
Bare multisig has one advantage over redeem/witness/leaf script multisig in that it's not necessary to backup all public keys/xpubs in order to reconstruct the spending script, the threshold number of private keys/xprvs is enough to recover funds.
Additionally, I don't think this change would really affect spam, as anyone determined to store raw data in unprunable outputs (as opposed to OP_RETURN or unused script branches) can just switch to a different standard output type while only sacrifi
...
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666333797)
Bare multisig has one advantage over redeem/witness/leaf script multisig in that it's not necessary to backup all public keys/xpubs in order to reconstruct the spending script, the threshold number of private keys/xprvs is enough to recover funds.
Additionally, I don't think this change would really affect spam, as anyone determined to store raw data in unprunable outputs (as opposed to OP_RETURN or unused script branches) can just switch to a different standard output type while only sacrifi
...
📝 maxwellcotto opened a pull request: "Create devcontainer.json"
(https://github.com/bitcoin/bitcoin/pull/28219)
<!--
*** Please remove the following help text before submitting: ***
Pull requests without a rationale and clear improvement may be closed
immediately.
GUI-related pull requests should be opened against
https://github.com/bitcoin-core/gui
first. See CONTRIBUTING.md
-->
<!--
Please provide clear motivation for your patch and explain how it improves
Bitcoin Core user experience or Bitcoin Core developer experience
significantly:
* Any test improvements or new tests that improv
...
(https://github.com/bitcoin/bitcoin/pull/28219)
<!--
*** Please remove the following help text before submitting: ***
Pull requests without a rationale and clear improvement may be closed
immediately.
GUI-related pull requests should be opened against
https://github.com/bitcoin-core/gui
first. See CONTRIBUTING.md
-->
<!--
Please provide clear motivation for your patch and explain how it improves
Bitcoin Core user experience or Bitcoin Core developer experience
significantly:
* Any test improvements or new tests that improv
...
✅ pinheadmz closed a pull request: "Create devcontainer.json"
(https://github.com/bitcoin/bitcoin/pull/28219)
(https://github.com/bitcoin/bitcoin/pull/28219)
💬 imacfan commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666352633)
Concept NACK
As a full node runner it's not acceptable to me to remove a data limit. Data limits are critical to making running a node accessible to as many as possible.
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666352633)
Concept NACK
As a full node runner it's not acceptable to me to remove a data limit. Data limits are critical to making running a node accessible to as many as possible.
💬 ns-xvrn commented on pull request "Remove arbitrary restrictions on OP_RETURN by default":
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666356763)
Concept NACK
Just because a protocol has unwanted steganography attacks, instead of deterring that - you're saying allow it with arms wide open. Totally unacceptable, this is not what Bitcoin protocol is for, as a full node operator - I don't want it.
(https://github.com/bitcoin/bitcoin/pull/28130#issuecomment-1666356763)
Concept NACK
Just because a protocol has unwanted steganography attacks, instead of deterring that - you're saying allow it with arms wide open. Totally unacceptable, this is not what Bitcoin protocol is for, as a full node operator - I don't want it.
💬 jonatack commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#discussion_r1284948058)
As a minimum, the tests would need to pass (if there is rough consensus on the change).
<details><summary>example test update</summary><p>
```diff
diff --git a/src/test/script_p2sh_tests.cpp b/src/test/script_p2sh_tests.cpp
index 739ab75de36..4e1225452fb 100644
--- a/src/test/script_p2sh_tests.cpp
+++ b/src/test/script_p2sh_tests.cpp
@@ -201,7 +201,7 @@ BOOST_AUTO_TEST_CASE(set)
{
SignatureData empty;
BOOST_CHECK_MESSAGE(SignSignature(keystore, CTransaction(t
...
(https://github.com/bitcoin/bitcoin/pull/28217#discussion_r1284948058)
As a minimum, the tests would need to pass (if there is rough consensus on the change).
<details><summary>example test update</summary><p>
```diff
diff --git a/src/test/script_p2sh_tests.cpp b/src/test/script_p2sh_tests.cpp
index 739ab75de36..4e1225452fb 100644
--- a/src/test/script_p2sh_tests.cpp
+++ b/src/test/script_p2sh_tests.cpp
@@ -201,7 +201,7 @@ BOOST_AUTO_TEST_CASE(set)
{
SignatureData empty;
BOOST_CHECK_MESSAGE(SignSignature(keystore, CTransaction(t
...
💬 cesarmassri commented on pull request "set `DEFAULT_PERMIT_BAREMULTISIG` to false":
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666362344)
> Bare multisig has one advantage over redeem/witness/leaf script multisig in that it's not necessary to backup all public keys/xpubs in order to reconstruct the spending script, the threshold number of private keys/xprvs is enough to recover funds.
>
> Additionally, I don't think this change would really affect spam, as anyone determined to store raw data in unprunable outputs (as opposed to OP_RETURN or unused script branches) can just switch to a different standard output type while only s
...
(https://github.com/bitcoin/bitcoin/pull/28217#issuecomment-1666362344)
> Bare multisig has one advantage over redeem/witness/leaf script multisig in that it's not necessary to backup all public keys/xpubs in order to reconstruct the spending script, the threshold number of private keys/xprvs is enough to recover funds.
>
> Additionally, I don't think this change would really affect spam, as anyone determined to store raw data in unprunable outputs (as opposed to OP_RETURN or unused script branches) can just switch to a different standard output type while only s
...