Make ui-keyring storage-agnostic

ui-keyring is just a wrapper around @polkadot/keyring which stores symmetrically-encrypted keys in localStorage. This can only work in web environment.

I’d propose to make it storage-agnostic, and create separately several transports:

• localStorage (for when security is not important, e.g. testnets)
• AsyncStorage (for React Native)
• file system (if used in node.js)

[jacogr]

Yes, there needs to be a different solution so you cannot “muck about” with the observable. apps also uses the observable directly, but just to “observe”, so subscribe and done.

Additions to FileStore is welcome, not sure it works. (But started typing and couldn’t stop.) The repo, as the name suggests, really is specific to the browser, and Electron is a browser as well. It is not too much of an issue to include that one in here since it basically needs fs, mkdirp - which generally is needed in other places as well for Node. So it shouldn’t break any real webpack-like env.

The issue with all these is that the footprint is so small that a package for the single handful of lines is just absolute complete overkill. However, the big drawback is that extra deps can be pulled-in. (i.e. every user now all of a sudden has a dep on extensionizer, and well, store as the default store.)

[polkadot-js-bot]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue if you think you have a related problem or query.