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

Update to support looking up mnemonic by just the first 4 letters of each word #16704

Merged
merged 14 commits into from
Nov 1, 2023

Conversation

cmmarslender
Copy link
Contributor

@cmmarslender cmmarslender commented Oct 27, 2023

Since the first 4 letters of each word is unique (or the full word, if less than 4 characters), and its common practice to only store the first 4 letters of each word in many offline storage solutions, this adds support to look up words by the first 4 characters

@cmmarslender cmmarslender requested a review from a team as a code owner October 27, 2023 05:30
@cmmarslender cmmarslender added the Added Required label for PR that categorizes merge commit message as "Added" for changelog label Oct 27, 2023
@cmmarslender cmmarslender marked this pull request as draft October 27, 2023 05:36
justinengland
justinengland previously approved these changes Oct 27, 2023
… the from short words function, which is always called prior to the old check
Copy link
Contributor

@altendky altendky left a comment

Choose a reason for hiding this comment

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

Seems like a good feature, thanks.

chia/util/keychain.py Outdated Show resolved Hide resolved
chia/util/keychain.py Outdated Show resolved Hide resolved
chia/util/keychain.py Outdated Show resolved Hide resolved
chia/util/keychain.py Outdated Show resolved Hide resolved
chia/util/keychain.py Outdated Show resolved Hide resolved
tests/core/util/test_keychain.py Outdated Show resolved Hide resolved
tests/core/util/test_keychain.py Outdated Show resolved Hide resolved
tests/core/util/test_keychain.py Outdated Show resolved Hide resolved
tests/core/util/test_keychain.py Outdated Show resolved Hide resolved
tests/util/bip39_test_vectors_short.json Outdated Show resolved Hide resolved
@cmmarslender cmmarslender marked this pull request as ready for review November 1, 2023 15:55
@cmmarslender cmmarslender merged commit 5343558 into main Nov 1, 2023
252 checks passed
@cmmarslender cmmarslender deleted the bip39-4-chars branch November 1, 2023 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Added Required label for PR that categorizes merge commit message as "Added" for changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants