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
HD Multisig derivation standard #1072
Merged
Merged
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
e6b9822
Create bip-0048.mediawiki
Fonta1n3 c9517ec
fixes
Fonta1n3 23d57cb
typo
Fonta1n3 42b9148
minor
Fonta1n3 ff04f6c
Update bip-0048.mediawiki
Fonta1n3 bfebc4b
Mention BIP 44 as the Multi-Account standard
ben-kaufman 4e81e16
Update bip-0048.mediawiki
Fonta1n3 9ec6bf6
Fix the table
ben-kaufman eae5288
Update bip-0048.mediawiki
Fonta1n3 3664283
Merge pull request #1 from ben-kaufman/patch-1
Fonta1n3 38096ce
remove bip44 stuff
Fonta1n3 86e7790
fix: remove legacy references
Fonta1n3 bf8c208
fix: define motivation, remove account creation blurb.
Fonta1n3 32d6ee2
fix: bip number not actually assigned
Fonta1n3 787ed87
Merge remote-tracking branch 'upstream/master'
Fonta1n3 dd3033f
fix: this is specific to an existing standard only
Fonta1n3 8a3a8bd
fix: update to provide for future extensibility
Fonta1n3 0b66777
fix: typo
Fonta1n3 1eb8a3c
Update bip-0048.mediawiki
Fonta1n3 c3c0abd
Update bip-0048.mediawiki
Fonta1n3 bc05931
Delete .DS_Store
Fonta1n3 20c5af5
Update bip-0048.mediawiki
Fonta1n3 90cb1c8
Update bip-0048.mediawiki
Fonta1n3 84e14b6
Update bip-0048.mediawiki
Fonta1n3 03f2d74
Fix BIP 48 headers and add to README
luke-jr File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have originally added the
script_type
level for forward compatibility. Specifically to avoid a BIP44/49/84 scenario, that is, to avoid creating a new BIP every time we wanted to use new type of scripts -- instead I thought it's easier to maintain a list of script types.from spesmilo/electrum#4352 (comment) :
There is another solution, which is to e.g. hash a descriptor or similar of the script type template to arrive at a derivation path level; then a single BIP would be enough; though that might make disaster-recovery problematic -- but this too requires its own
script_type
level.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, arguably there is no backward compatibility provided at all. What exactly is this backward compatible with?
I guess you mean the wallets that are using this same spec before it was formally specced (here), but as you don't add anything new, just formalise it, that is just self-referential.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, I have added some simple text around future script types and extensibility in 8a3a8bd, does that work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, thanks. I personally like this approach better; though I guess it might be subjective. (re new BIP every time vs maintaining list of script types.) I prefer the wording to express the original intention, to explain why it was done like this.