Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feat]: mempoolfullrbf=1 by default #142

Closed
1 task done
petertodd opened this issue Mar 29, 2024 · 4 comments · Fixed by #143
Closed
1 task done

[feat]: mempoolfullrbf=1 by default #142

petertodd opened this issue Mar 29, 2024 · 4 comments · Fixed by #143

Comments

@petertodd
Copy link

Prerequisites

  • I have searched for existing issues that already suggest this feature.

Describe the Feature Request

Since about 90%+ of hashpower is mining full-rbf, it makes sense to accept and relay those replacements.

Describe the Use Case

Failing to accept and relay full-RBF replacements when almost all hash power accepts them decreases the accuracy of mempool fee estimation, and prevents users from learning about relevant transactions that are likely to get mined. This is particularly notable for users who are running block explorers on their Start 9 devices; why block explorer sites like https://mempool.space and https://blockstream.info have supported full-RBF for quite some time now.

Describe Preferred Solution

Just set mempoolfullrbf=1 by default; the configuration tool can continue to allow it to be disabled if desired. Bitcoin Core doesn't seem interested in doing this by default — likely politics around how controversial it has been. But there's no reason why we should not do the right thing for users.

Describe Alternatives

No response

Anything else?

No response

@kn0wmad
Copy link
Contributor

kn0wmad commented Mar 29, 2024

As you noticed, we have generally stuck to Core defaults in an attempt to remain neutral. However, I'm sure this is worth a discussion.

@Dominion5254 Dominion5254 linked a pull request Apr 10, 2024 that will close this issue
@remcoros
Copy link

Contrary to my earlier statements (in another PR) about sticking with Core defaults. I think this one makes a lot of sense for (non-technical) users of 'full node providers' like StartOS and Umbrel.

Users of node distributions like this, do not normally follow the Bitcoin Core development, announcements and/or forums and completely rely on 'sane' defaults from these node providers.

Since it seems the network and users (not the core devs) have reached consensus about enabling mempoolfullrbf, and it improves end-user experience (fee estimation), this change makes sense to me.

Ack

@k0gen
Copy link
Member

k0gen commented Apr 11, 2024

v26.1.0 will have mempoolfullrbf=1 as default

@petertodd
Copy link
Author

petertodd commented Apr 11, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants