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

Non-cryptographic hashes #30

Closed
jbenet opened this issue Jun 14, 2016 · 5 comments
Closed

Non-cryptographic hashes #30

jbenet opened this issue Jun 14, 2016 · 5 comments

Comments

@jbenet
Copy link
Member

jbenet commented Jun 14, 2016

What do we do about non-cryptographic hashes, such as these? Do we include them in the same table?

I'm leaning towards yes.

@jbenet
Copy link
Member Author

jbenet commented Jun 14, 2016

I'm leaning towards yes because:

  • we already plan on including "the identity hash function", which is not cryptographic.
  • it is better than creating another table for a different but equivalent standard.

@RichardLitt
Copy link
Member

It is better than creating another table for a different but equivalent standard.

I agree with this.

@jbenet
Copy link
Member Author

jbenet commented Aug 14, 2016

So far -- after much discussion with relevant people -- it looks like we will start adding non cryptographic hashes. The well-known hash functions are not more than 16000, and thus 2 bytes are plenty. This is a small price to pay for not having two standards or two different standard tables.

@jbenet
Copy link
Member Author

jbenet commented Aug 14, 2016

I'll give a week for people to voice reasonable discontent.

@jbenet
Copy link
Member Author

jbenet commented Sep 17, 2016

Okay, we'll do this

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

2 participants