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

Crypto appendix - what about SHA-512/224? #2448

Open
randomstuff opened this issue Dec 9, 2024 · 4 comments
Open

Crypto appendix - what about SHA-512/224? #2448

randomstuff opened this issue Dec 9, 2024 · 4 comments
Labels
1) Discussion ongoing Issue is opened and assigned but no clear proposal yet AppendixV Appendix with crypto details _5.0 - Not blocker This issue does not block 5.0 so if it gets addressed then great, if not then fine.

Comments

@randomstuff
Copy link
Contributor

The section of the crypto appendix about hash functions mentions SHA-512/256 (approved) but does not mention SHA-512/224 in either "Approved Hash Functions for Password Storage" or "Disallowed Hash Functions".

I think it is weird that is is not mentioned in either section.

SHA-512/224 is mentioned in other places:

  • "Disallowed Hash Functions for Digital Signatures"
  • "Disallowed Hashes for RBG"

FWIW,

Proposition: can we just include it in "Disallowed Hash Functions"?

@randomstuff
Copy link
Contributor Author

ping @danielcuthbert

@tghosth tghosth added 1) Discussion ongoing Issue is opened and assigned but no clear proposal yet _5.0 - Not blocker This issue does not block 5.0 so if it gets addressed then great, if not then fine. AppendixV Appendix with crypto details labels Dec 10, 2024
@unprovable
Copy link

We included SHA512/256 as, although it is a truncated hash, it still provides a large output that is likely going to remain resistant to truncated hash collision attacks. We set the bar at 256bits in length to ensure this, hence we did not allow SHA512/224 - in short, it's too short for where we drew the line.

You can probably include it, but I wouldn't make a point of it. In our analysis internally at a large enterprise, it's barely used. So our calculation was; it's not very long + it's a truncated hash + it's barely used = don't include it.

@randomstuff
Copy link
Contributor Author

it's not very long + it's a truncated hash + it's barely used = don't include it

Do you mean "don't approve it" or "don't talk about it"?

To be clear, my proposition was to include in "Disallowed Hash Functions" (not allow it). Not saying whether it is approved or not feels weird. Maybe it would make sense to include it as "legacy/discouraged" (see #2398)?

@unprovable
Copy link

Ah I understand - yes, remove it if possible, certainly consign it to "legacy/disallowed for new designs". Apologies for the mix up, M.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1) Discussion ongoing Issue is opened and assigned but no clear proposal yet AppendixV Appendix with crypto details _5.0 - Not blocker This issue does not block 5.0 so if it gets addressed then great, if not then fine.
Projects
None yet
Development

No branches or pull requests

3 participants