Currently every Genesis VM uses a single encryption key for every asset stored and used by that VM. While every VM uses a unique global key, security of the assets could be improved by using unique encryption keys for every asset in every VM. This is exactly what this PR does.
The second commit isn't necessary per say, after making the changes necessary to allow every asset to use a unique key, I realized the GensisVM.DecryptionKeys member was unnecessary. I don't know if you guys want to keep that data member around for debugging purposes, if you do I'll revert that last commit and we'll be good to go.
Lastly, this project is structured and laid out very logically and nicely, making changes like this is easy. Thanks for creating such a great project that is also so easy to maintain!
Currently every Genesis VM uses a single encryption key for every asset stored and used by that VM. While every VM uses a unique global key, security of the assets could be improved by using unique encryption keys for every asset in every VM. This is exactly what this PR does.
The second commit isn't necessary per say, after making the changes necessary to allow every asset to use a unique key, I realized the GensisVM.DecryptionKeys member was unnecessary. I don't know if you guys want to keep that data member around for debugging purposes, if you do I'll revert that last commit and we'll be good to go.
Lastly, this project is structured and laid out very logically and nicely, making changes like this is easy. Thanks for creating such a great project that is also so easy to maintain!