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

Payment support for confidential addresses #86

Conversation

sekulicd
Copy link
Collaborator

@sekulicd sekulicd commented Jun 8, 2020

Before there was no option to create p2pkh and p2sh confidential addresses using payment package.

After the methods ConfidentialPubKeyHash/ConfidentialScriptHash are added, it is possible to convert create confidential p2pkh and p2sh addresses.

It closes #83 .
It closes #85.

Please @altafan can you review this?

go.sum Outdated Show resolved Hide resolved
payment/payment.go Outdated Show resolved Hide resolved
payment/payment_test.go Outdated Show resolved Hide resolved
payment/payment.go Outdated Show resolved Hide resolved
payment/payment.go Outdated Show resolved Hide resolved
payment/payment.go Outdated Show resolved Hide resolved
payment/payment_test.go Show resolved Hide resolved
payment/payment_test.go Outdated Show resolved Hide resolved
payment/payment.go Outdated Show resolved Hide resolved
@altafan altafan changed the title Payment support for blinding key in base58 addresses Payment support for confidential addresses Jun 9, 2020
Copy link
Member

@tiero tiero left a comment

Choose a reason for hiding this comment

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

Could you add confidential examples as documentation in examples_test.go? So we can have it here https://pkg.go.dev/github.com/vulpemventures/go-elements@v0.0.1/payment?tab=doc

Copy link
Collaborator

@altafan altafan left a comment

Choose a reason for hiding this comment

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

LGTM but, please, fix the comments

payment/payment.go Outdated Show resolved Hide resolved
payment/payment.go Outdated Show resolved Hide resolved
payment/payment.go Show resolved Hide resolved
payment/payment.go Show resolved Hide resolved
payment/payment.go Outdated Show resolved Hide resolved
payment/payment.go Outdated Show resolved Hide resolved
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.

[Payment] Support for blech32 confidential addresses [Payment] Support for base58 confidential addresses
3 participants