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

Add support for bech32m bitcoin wallets #2873

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

sstone
Copy link
Member

@sstone sstone commented Jul 3, 2024

These changes allow eclair to be used with a bitcoin core wallet configured to generate bech32m (p2tr) addresses and change addresses, which is not possible currently because there are a few place where p2wpkh is "hardcoded" (i.e. we implicitly assume that wallet addresses are p2wpkh addresses). We also handle the special case where eclair manages bitcoin core's private keys.
The wallet still needs to be able to generate bech32 (p2wpkh) addresses in some cases (support for static_remote_key for non anchor channels for example).

@sstone sstone force-pushed the use-bech32m-wallet branch from 9f9774f to d603e95 Compare July 17, 2024 08:20
@sstone sstone force-pushed the use-bech32m-wallet branch from d603e95 to 2353b24 Compare August 23, 2024 09:33
@sstone sstone force-pushed the use-bech32m-wallet branch 2 times, most recently from cd2dac0 to 02c9ab2 Compare September 11, 2024 07:39
@sstone sstone force-pushed the use-bech32m-wallet branch 2 times, most recently from 6f067ff to eb4b6c7 Compare October 8, 2024 16:43
These changes allow eclair to be used with a bitcoin core wallet configured to generate bech32m (p2tr) addresses and change addresses.
The wallet still needs to be able to generate bech32 (p2wpkh) addresses in some cases (support for static_remote_key for non anchor channels for example).
@sstone sstone force-pushed the use-bech32m-wallet branch from eb4b6c7 to 39e2842 Compare December 9, 2024 17:18
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 this pull request may close these issues.

1 participant