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

cipher suites #127

Merged
merged 6 commits into from Jun 18, 2021
Merged

cipher suites #127

merged 6 commits into from Jun 18, 2021

Conversation

emanjon
Copy link
Collaborator

@emanjon emanjon commented Jun 1, 2021

No description provided.

@gselander
Copy link
Collaborator

Looks good. One inconsistency is that there is no cipher suite with AES-GCM and NIST curves, like (A128GCM, SHA-256, P-256, ES256, A128GCM, SHA-256).

This could be a new cipher suite 5. Then all non-CNSA suites would come in pairs.

@emanjon
Copy link
Collaborator Author

emanjon commented Jun 2, 2021

Looks good. One inconsistency is that there is no cipher suite with AES-GCM and NIST curves, like (A128GCM, SHA-256, P-256, ES256, A128GCM, SHA-256).

This could be a new cipher suite 5. Then all non-CNSA suites would come in pairs.

That is very intentional. The GCM cipher suite uses X25519 for key exchange and ECDSA for authentication. This seems to be the dominant choice for HTTPS on the Web. Unless you are constrained you typically use X25519 for key exchange for speed and ECDSA for authentication as that is what the CAs support.

For an IoT device you probably want to have a single curve implemented. Also it might be much easier to switch to EdDSA as you might not have to rely on Global CAs and all other HTTPS nodes to support EdDSA. You can just deploy you own CWT server and make sure the nodes in you new IoT deployment support EdDSA.

I have not heard any IoT people very interested in GCM. People seems far more interested in ChaCha20.

@emanjon
Copy link
Collaborator Author

emanjon commented Jun 17, 2021

Ready to merge?

@gselander gselander merged commit b7ea127 into master Jun 18, 2021
@emanjon emanjon deleted the cipher-suites branch August 19, 2021 14:54
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.

None yet

2 participants