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

[Unblinding UTXO] Ability to detect and spend unblinded addresses #36 #39

Draft
wants to merge 3 commits into
base: trunk
Choose a base branch
from

Conversation

claddyy
Copy link

@claddyy claddyy commented Oct 9, 2024

Addresses #36
This PR works on adding the solutions from Blockstream/lwk#38 (comment)

Comment on lines 467 to 470
let blinding_key = wallet.blinding_key().unwrap();
let unblinded_address = "";
let unblinded_address = println!("The unblinded address is {}", unblinded_address);
println!("Fund the unblinded address");
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll have to create a function that converts unblinded_address to blinded using the binding key.
I couldn't find any method that does this either in lwk-dart repo or in lwk.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you mean blinded to unblinded? We don't need to go the other way around.

let unblinded_address = confidential_address.standard;

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This struct has methods to go either direction:

https://docs.rs/elements/0.24.1/elements/address/struct.Address.html

Comment on lines 458 to 459
let electrum_url = "les.bullbitcoin.com:995".to_string();
wallet.sync(electrum_url.clone()).unwrap();
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we're on testnet now, I'll change this to use blockstream testnet url.

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.

2 participants