-
Notifications
You must be signed in to change notification settings - Fork 502
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
Breaking cleanup upgrade: 3.0.0 #134
Comments
Big plus for the hex format issue. |
Does 'Encryption keys in the keystore' include removing the multiEncryptString / multiDecryptString functions? We use those a fair bit in our app. |
This is great, we use this library and think it is fantastic. Glad to see it is still being supported. We rely on serialize/deserialize the keystore to and from localStorage, and also the client side tx signing is super useful. |
@jaycarey Good to know, thanks! The plan was to remove the explicit handling of encryption keys in the keystore itself. What I'll probably end up doing is to modify the encryption module to use a raw private key. That way you can still use it by exporting private keys from the keystore. |
just wanted to add that we rely on serializing/deserializing the keystore to and from a db, seems like a pretty big breaking change if that were to be removed |
I have found the time to finish this up, will try to put up a pull request and merge in this today. Note that you'll need to update the following:
@tzapu Serialization and deserialization is not removed. |
This is now merged: #166 |
Returning from a bit of an absence from Lightwallet I'm going to do a long overdue cleanup/refactoring release. Things that I intend to address are:
Cruft/unused stuff
These things I don't think are used very much, makes the code bloated and complex and I intend on removing them:
createVault()
)Updated hex formats
When lightwallet was created it was unclear if addresses should have a
0x
prefix or not, so we opted to use without0x
. Now it's clear that most applications use0x
and this is even required in some places so we'll be switching to using0x
exclusively.Feedback
If you have any other changes that you would like to see, please let us know here 😃
The text was updated successfully, but these errors were encountered: