Possible vulnerability on the node unique ID

[brunoerg]

MD5 is used on the source code to create the node unique ID, however, md5 is a vulnerable algorithm (collision attacks). To improve the security of the protocol, I suggest to change it to a secure and efficient algorithm (Blake2, for example).

[macsnoeren]

You are correct. I do a simple ID creation and MD5 will most probably not have any collisions in this particular case. This code is used for my Blockchain course, so I will not change it now. I will consider your suggestion during the new class.

[macsnoeren]

I have changed the md5 to sha512 to create the id within the new class. So less likely to have collision attacks. Thanks for your comment.