-
-
Notifications
You must be signed in to change notification settings - Fork 679
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
6.2.9 and CCM-8 #2485
Comments
Seems like getting rid of it is the right way to go. |
@randomstuff can you prepare a PR that captures the feedback from this issue and also #2143. Does using the disclaimer from here help with this issue? It sounds like they did have a reason for including it so I would be cautious about reversing it. |
Ping @randomstuff :) |
Woops sorry, I'll do that! By the way, I think you linked to the wrong issue in your previous comment :) |
@unprovable, any opinion on this? |
I searched https://clienttest.ssllabs.com:8443 and did not find a client with support for this (is it supported by the TLS client test?). I made a PR to disapprove it. I believe it won't be missed. If anyone thinks it should be approved, please comment here. |
Since #2482, 6.2.9 is:
CCM-8 is still listed in the approved algorithms. However, its authentication tag only has 64 bits of security and does not respect this minimum of 128-bits of security.
There seems to be a conflict between these. How should be fix this?
Possible options:
I don't have a strong opinion on this and I don't know if CCM-8 is really used/useful nowadays. The safest choice seems to disapprove CCM-8. Unless/until someone come forwards with further input, I would suggest taking this approach.
See as well discussion in #2413 for more info about CCM-8.
The text was updated successfully, but these errors were encountered: