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

Fix bits length of Shake 128 (16 not 32) & 256 (32 not 64) #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vorburger
Copy link

@ianopolous @mk868 does this seem better to you as well? (Or am I just confused?)

@odisseus
Copy link
Contributor

odisseus commented Jan 4, 2025

These are extendable-output algorithms that can produce hashes of any length. As far as I understand, the outputs of SHAKE128 and SHAKE256 do indeed have just 16 and 32 bytes of entropy, but nothing prevents you from generating a SHAKE128 hash that's 1000 bytes long.

That said, it looks like this library doesn't support hashes of variable length (see #45).

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

Successfully merging this pull request may close these issues.

2 participants