swaponline / MultiCurrencyWallet

Bitcoin, Ethereum, ERC20 crypto wallets with 0x and atomic swap exchange.
https://swaponline.github.io
MIT License
468 stars 488 forks source link

Add BIP 0038 support (Passphrase-protected private key) #2696

Open noxonsu opened 4 years ago

noxonsu commented 4 years ago

добавить возможность зашифровать приватник паролем. для этого добавить поддержку этой либы https://github.com/bitcoinjs/bip38

простейший способ

noxonsu commented 4 years ago

кстати если зайти в https://github.com/bitcoinjs/bip38/blob/master/package.json вот тут можно посмотреть как ее юзают другие прогеры, можно спокойно тырить с опенсорса. например

vladiuz1 commented 4 years ago

Это противоречит концепту derived wallets у которого приватник генерится по sequence

Если делать так внутри концепта то тогда все кошельки должны быть защищены паролем а это будет очень тяжко

noxonsu commented 4 years ago

ну надо понять как остальные делают. https://iancoleman.io/bip39/ вообще bip39 и bip38 совместимы но тормозит жуть. нужно смотреть как другие делают, например электрум на диске хранит сид, истторию, приватники и все адреса, если активировать пароль он просто шифрует файл с базой. я думаю можно так же делать с локалстораджем. пароль нам нужен это требования bitcoin.org https://github.com/swaponline/MultiCurrencyWallet/issues/2635

я думаю можно сделать наиболее просто, далее у них на ревью пусть уточняют

vladiuz1 commented 4 years ago

Ну мы там не проходим и по другим параметрам

У нас нет authentication по замыслу

vladiuz1 commented 4 years ago

Просто оставте эту идею, мы не пройдем все равно

А шифровать сейчас начать приватники это сломает всю концепцию