Skip to content
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

MD5 #50

Closed
jw3126 opened this issue Jan 11, 2018 · 10 comments
Closed

MD5 #50

jw3126 opened this issue Jan 11, 2018 · 10 comments

Comments

@jw3126
Copy link

jw3126 commented Jan 11, 2018

I have a md5 implementation here. It uses the same structure as SHA.jl and I copy pasted some SHA.jl code there. This is not ideal, I can think of a couple of other options:

  • Merge MD5.jl into SHA.jl
  • Create some HashFunctionsBase.jl package with common code
  • Let MD5.jl depend on SHA.jl

What do you think @staticfloat ?

@oxinabox
Copy link
Member

Hah, we actually did this at the same time (like to the hour I think).
See: https://github.com/oxinabox/MD5.jl
#46

I did the "Let MD5.jl depend on SHA.jl"

@jw3126
Copy link
Author

jw3126 commented Jan 11, 2018

Did you look at the performance? I got rid of all branches in tranform!, but libnettle is still twice as fast.

@oxinabox
Copy link
Member

I did not.
Yours looks more performant than mine.
I wanted to get it done with minimum code.

I think the most performant (and also cleanest) would be somewhere between the two.
It'll take some tweaking to find.

@jw3126
Copy link
Author

jw3126 commented Jan 11, 2018

We can discuss it here JuliaCrypto/MD5.jl#2

@staticfloat
Copy link
Member

Just FYI, there is a new JuliaCrypto organization that we're starting to collect packages underneath.

@jw3126
Copy link
Author

jw3126 commented Apr 16, 2020

@oxinabox shall we move MD5 to JuliaCrypto?

@oxinabox
Copy link
Member

We should, but I can't seem to find my invite. @ViralBShah are you sure you sent the invite to the org right?

@ViralBShah
Copy link
Member

Yeah shows me as pending. Maybe check your spam? Is there a way to see on GitHub itself?

@oxinabox
Copy link
Member

Ah, if i go to the org page myself I can see it.
Sorted

@oxinabox
Copy link
Member

Done
https://github.com/JuliaCrypto/MD5.jl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants