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 blinding pegins #663

Merged
merged 5 commits into from
Jun 21, 2019

Conversation

stevenroose
Copy link
Member

@stevenroose stevenroose commented Jun 12, 2019

Adds pegin handling support for blindrawtransaction and rawblindrawtransaction.

@stevenroose stevenroose force-pushed the confidential-pegins branch from 7398e0b to 03388cc Compare June 12, 2019 21:46
@stevenroose stevenroose changed the title WIP: Add support for blinding pegins Add support for blinding pegins Jun 13, 2019
@stevenroose stevenroose requested a review from instagibbs June 13, 2019 11:00
test/functional/feature_fedpeg.py Outdated Show resolved Hide resolved
test/functional/feature_fedpeg.py Outdated Show resolved Hide resolved
test/functional/feature_fedpeg.py Show resolved Hide resolved
@instagibbs
Copy link
Collaborator

looks good otherwise, and nice to see the python stuff put to work :P

@stevenroose stevenroose force-pushed the confidential-pegins branch from 6115878 to a22c76d Compare June 13, 2019 16:03
@stevenroose stevenroose force-pushed the confidential-pegins branch from a22c76d to 56358b8 Compare June 13, 2019 16:04
@stevenroose
Copy link
Member Author

and nice to see the python stuff put to work :P

Yeah nice to see that we can do all that stuff manually. I think the only thing that we can't do without liquid is the blinding, but that should be possible in Rust with rust-secp256k1-zkp. Very excited about that!

@stevenroose
Copy link
Member Author

@instagibbs build is fine it seems

@instagibbs
Copy link
Collaborator

re:"Always return blind address for "blech32" type"

The help for getnewaddress should reflect this.

@instagibbs
Copy link
Collaborator

utACK cc49106

@instagibbs instagibbs merged commit cc49106 into ElementsProject:master Jun 21, 2019
instagibbs added a commit that referenced this pull request Jun 21, 2019
cc49106 Update getnewaddress help message with "blech32" special case (Steven Roose)
56358b8 Add tests for blinding pegin transactions (Steven Roose)
a9f589b Add support for blinding pegin transactions (Steven Roose)
bf0cc83 Add confidential assets values to listunspent documentation (Steven Roose)
4e84596 Always return blind address for "blech32" type (Steven Roose)

Pull request description:

  Adds pegin handling support for `blindrawtransaction` and `rawblindrawtransaction`.

Tree-SHA512: 1138aeedd2d5852108abd53a6afc83524e5f06ad3ab770fd8ab37c97c8dd5c49f78b8d44bdf7cbe2c073e07cd8566bf032526ab6488e99fc02f8b5b154a3fde1
@stevenroose stevenroose deleted the confidential-pegins branch February 2, 2020 14:36
apoelstra added a commit to apoelstra/elements that referenced this pull request Nov 3, 2020
apoelstra added a commit to apoelstra/elements that referenced this pull request Nov 9, 2020
gwillen pushed a commit that referenced this pull request Mar 3, 2021
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