Skip to content

Wordlist Portuguese Brazil BIP39#385

Merged
NicolasDorier merged 1 commit intoMetacoSA:masterfrom
alissonsolitto:wordlist-ptbr
Apr 7, 2018
Merged

Wordlist Portuguese Brazil BIP39#385
NicolasDorier merged 1 commit intoMetacoSA:masterfrom
alissonsolitto:wordlist-ptbr

Conversation

@alissonsolitto
Copy link
Copy Markdown
Contributor

Implementation of the list of words in the Portuguese-Brazilian language.
The word list was obtained at this link:

Issue: bitpay/bitcore-mnemonic#52
WordList: https://github.com/bitpay/bitcore-mnemonic/files/559361/brazilian_english.js.zip

Credits for the word list: https://github.com/eddieoz

@NicolasDorier
Copy link
Copy Markdown
Collaborator

Nice!

@NicolasDorier NicolasDorier merged commit 5e276cd into MetacoSA:master Apr 7, 2018
@johnnyasantoss
Copy link
Copy Markdown

johnnyasantoss commented Jun 11, 2021

The list created by @eddieoz isn't the one set by the BIP39 (see bitcoin/bips#998). Actually, at the time that this PR was merged, there wasn't any pt-BR lists that were approved by the BIP39 proposal.

I just hope no one used this word list.

EDIT: the issue (bitpay/bitcore-mnemonic#52 that should have been an PR) was never merged/done.

@alissonsolitto
Copy link
Copy Markdown
Contributor Author

@johnnyasantoss

Very well pointed out, I stayed out of the crypt world until then.

@johnnyasantoss
Copy link
Copy Markdown

Well, let's get this fixed. I'm just not sure how it would be the transition process from one word list to another. Maybe a program that exchanges the words based on their index, but that might not be the best ux.

@NicolasDorier
Copy link
Copy Markdown
Collaborator

wow ok I will use the new version

@NicolasDorier
Copy link
Copy Markdown
Collaborator

@johnnyasantoss I replace the old list with the new one. What I can do if somebody need it is to keep the old list and call it "legacy". Then end wallet can just check the current wordlist, and if checksum fail, try to fallback on the legacy one.

@johnnyasantoss
Copy link
Copy Markdown

@NicolasDorier great! I'll be looking out for other bitcoin libs to check if they also have the non-standard wordlist.

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.

3 participants