search

dmazzella / ucrypto

Micropython package for doing fast rsa and elliptic curve cryptography, specifically digital signatures
MIT License
33 stars 11 forks source link

hashlib.md5() seems to be removed in latest uhashlib #8

Closed y3k00000 closed 1 year ago

y3k00000 commented 1 year ago

As title, found this issue when installing .py files to my pico_w.

I currently comment out the occurrences of hashlib.md5 to make it work.

y3k00000 commented 1 year ago

from Document

MD5 - A legacy algorithm, not considered cryptographically secure. Only selected boards, targeting interoperability with legacy applications, will offer this.

I think this is another board supporting issue.

MATTYGILO commented 1 year ago

Hi @y3k00000 have you yet gotten this to work, I tried this repo back in the day. It seemed to not work at all even if you compiled it and put it on a device?

y3k00000 commented 1 year ago

@MATTYGILO After the u2f was built and uploaded to the board you'll need to manually copy files from Here into the board's /lib directory.

If hashlib on your board doesn't provide the md5() then you'll have to edit the copied py files and comment out all md5() related calls.

Then you can begin importing the module and see if it's working. image

dmazzella commented 1 year ago

solved in eba2c08

dmazzella commented 1 year ago

@y3k00000 if you can confirm that it works we can close the issue, tnx

y3k00000 commented 1 year ago

@dmazzella The importing is ok now. 👍 image

But when I tried running ecdsa test image

It seems a further fix is required.

dmazzella commented 1 year ago

fixed in ffbd73

y3k00000 commented 1 year ago

Tested and it's ok now.