Skip to content

Conversation

kallewoof
Copy link
Contributor

@kallewoof kallewoof commented Jan 26, 2022

…addresses via descriptor calls.

This is not obvious, so deserves some documentation. It might be worth it to also add a 'don't care about checksum' flag to deriveaddresses to avoid the getdescriptorinfo step.

See #24160, #24162.

Copy link
Contributor

@shaavan shaavan left a comment

Choose a reason for hiding this comment

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

Concept 0 in favor of #24162

As explained by @kallewoof here, the private key is in WIF format, which has checksum included in it. This two-step behavior of first deriving the checksum and then the addresses is hence redundant.

So I think instead of providing documentation for the current behavior, its better to fix this redundancy which is being done in #24162

@DrahtBot
Copy link
Contributor

DrahtBot commented Jan 26, 2022

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

No conflicts as of last run.

@luke-jr
Copy link
Member

luke-jr commented Jan 31, 2022

Concept NACK. There's really no good reason to use the same private key for multiple different addresses, is there?

@kallewoof
Copy link
Contributor Author

kallewoof commented Jan 31, 2022

There are good reasons to use the same privkey for multiple different addresses. One of them is unit testing / test vectors for some application that uses addresses (e.g. BIP-322).

Edit: besides, all this does is describe how to do it, the functionality itself is already there.

@kallewoof kallewoof force-pushed the 202201-derivefromprivkey branch from 704eea8 to 523e51f Compare May 6, 2022 06:19
@kallewoof kallewoof closed this Apr 11, 2023
@kallewoof kallewoof deleted the 202201-derivefromprivkey branch April 11, 2023 04:43
@bitcoin bitcoin locked and limited conversation to collaborators Apr 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants