threefoldtecharchive / rivine

Blockchain technology for creating custom chains.
Apache License 2.0
22 stars 12 forks source link

Unencrypted wallet support #345

Closed robvanmieghem closed 5 years ago

robvanmieghem commented 6 years ago

Wallets running inside a container currently need a real shell to paste the password. This is often realized by using an ubuntu container to start from and the password is found in scripts/logs anyway. A simple container with only the daemon and client binaries is preferred

GlenDC commented 6 years ago

I propose the following high-level approach:

We do not define any logic/feature to convert a non-encrypted wallet to an encrypted wallet. It could be supported should be needed, and it would be very simple to do given my suggested approach, but given the motivation for the feature I do not see why that would be needed, hence better not to spend our energy on this.

GlenDC commented 6 years ago

Started working on this, but going to pause the dev on this, and instead focus on

Given that I might want to change some small things in the (Rivine) wallet code, in order to allow unencrypted wallets. While they may be small, it probably will make the merging with the current Sia wallet code a bit more difficult, hence why I'll put this issue on hold for now.