-
-
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
Cryptography tweak wording related to "authenticated encryption", "MAC algorithm" #2495
Comments
My suggestions and comments:
OK. It feels somewhat weird to explicitly state "authenticated encryption" at this point but I guess it works.
I would not include "authenticated" here because we want to identify all instances of cryptography, even if they are not authenticated encryption :)
OK.
I would suggest "V6.5 Encryption" instead. The requirements in this section apply everytime you want to use encryption and one of these requirements should be to use authenticated encryption.
OK. |
I am extremely grateful that Bart is providing this feedback. Any chance we can get these into 5.0? |
I think it is unnecessary, we are talking conceptually here.
Agree
I propose Encryption at Rest as later on we talk about "In-Use Data Cryptography" I agree with the other changes. |
Why "at rest"? This applies equally to in transit encryption (TLS, DTLS, etc.). |
because we talk about encryption in transit in V9 |
Yes, however these requirements (V6.5) could (should ?) apply equally to encryption in transit (and are not "duplicated" in V9). I understood that these requirements (in V6.5) were intended to apply to in-transit encryption (TLS) as well. Do you think these requirements should not apply to TLS and such? |
For reference, here is the feedback from Bart Preneel on this topic (to clarify, this was a comment which we have received previously, not related to this specific exchange):
I was intending to file a separate issue but this is related to this one so maybe we can discuss all these points here? |
So maybe just change the heading to "Encryption Algorithms" ? Would that be sufficient? |
Yes that, looks good to me. |
Ok thanks, I opened #2527 to handle this |
We have received this feedback from Bart Preneel.
Current:
Proposed:
Current:
Proposed:
Current:
Proposed:
Current:
Proposed:
The text was updated successfully, but these errors were encountered: