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.

Sign up for GitHub

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

Jump to bottom

Switch uniquehasher to crc32c #1

Closed

fasterthanlime opened this issue Jan 25, 2016 · 1 comment

Collaborator

fasterthanlime commented Jan 25, 2016

We don't need cryptographic hashes, it's about data integrity

fasterthanlime changed the title ~~Switch uniquehasher to crc32~~ Switch uniquehasher to crc32c

Collaborator Author

fasterthanlime commented Jan 26, 2016

We don't need cryptographic hashes, it's about data integrity

That's wrong — actually the weak hash is a CRC32 variant

fasterthanlime closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment