-
Notifications
You must be signed in to change notification settings - Fork 234
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
Merkle Hashing #19
Comments
it should be SHA-256. |
Thanks. |
we should implement as interface,make it easy to change
…Sent from my iPhone
On Jan 5, 2018, at 19:46, EanCusnaivy ***@***.***> wrote:
Thanks.
Already changed to SHA-256 yesterday in new PR.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Wouldn't SHA-3 / Keccak be a better alternative? |
yes, sure , but interfaces are better |
Good |
Totally agree |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I see that you use MD5 for merkle hashing, why not SHA?
See: https://github.com/AElfProject/AElf/blob/ff73574afaf9ecf64f98ff472da4247c6586a209/AElf.Kernel/ExtensionMethods.cs
The text was updated successfully, but these errors were encountered: