-
Notifications
You must be signed in to change notification settings - Fork 115
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
SHA-512/256 in the documentation might result a bit ambiguous #101
Comments
Want to take a stab at disambiguating this in on the website (PR against: https://github.com/multiformats/website/blob/master/content/multihash.md)?
|
FIPS 180-4 (section 5.3.6) actually defines a generic way to have |
File one against multiformats/multicodec. We really should remove the table here. Pick something in the 2-3 byte range. FYI, such a PR may take a while to get merged as it'll take time to review and isn't really a priority (although simply having a PR open will help us track progress). Just wanted to warn you. |
Can the same issue be applied to |
We can but we'll need to define a range, again. Also, does anyone actually use sha3-512/k? I'm really not seeing why these specs aren't just truncating hashes. |
Your homepage implies in its examples that "SHA-512 (256 bit)" is the truncated version of SHA-512, and that's pretty clear in the
0x1320
header, but since there are a few instances (e.g. BouncyCastle's"SHA-512/256"
cipher, also BSD tool/sbin/sha512t256
) that use a similar name for FIPS 180-4 implementation of SHA-512/t, I would suggest adding a note regarding that (and, maybe, even add that cipher to the multihash cipher table with his own number?).“Simply truncated” SHA-512 hash, as in your current documentation:
52eb4dd19f1ec522859e12d89706156570f8fbab1824870bc6f8c7d235eef5f4
FIPS 180-4 implementation of SHA-512/t:
006fff7ca0bd5b4a5b01706525ca739e63bf9dbdced6da91911d71b42667ba7f
The text was updated successfully, but these errors were encountered: