⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@ejohnstown
Copy link
Contributor

@ejohnstown ejohnstown commented Jan 14, 2026

  1. Add the KEX algorithms mlkem1024nistp384-sha384 and mlkem768x25519-sha256.
  2. Performed some comment and whitespace maintenance.
  3. Updated the KEX test to test any KEX given the string with its name. Added the new KEX algorithms to the test.

This solves issue #757.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds support for two new ML-KEM (post-quantum) hybrid key exchange algorithms to wolfSSH: mlkem1024nistp384-sha384 and mlkem768x25519-sha256. These algorithms combine traditional elliptic curve cryptography with post-quantum ML-KEM for enhanced security against future quantum computing threats.

Changes:

  • Added support for two new hybrid KEX algorithms combining ML-KEM with traditional elliptic curves
  • Refactored the KEX test infrastructure to accept algorithm names as parameters
  • Updated documentation and comments for the new algorithms

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 7 comments.

File Description
wolfssh/internal.h Added conditional compilation flags, algorithm IDs, buffer size updates, and HandshakeInfo bitfield for the new ML-KEM algorithms
src/internal.c Implemented client and server key agreement logic for the new algorithms, updated algorithm name mappings, hash functions, and prime curve lookups
src/ssh.c Updated display text for ML-KEM algorithm identification
tests/kex.c Refactored test to be algorithm-agnostic and added tests for the two new KEX algorithms

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

anhu and others added 2 commits January 13, 2026 21:54
1. Reorganize the KEX test to allow for testing any KEX algorithm. Add test
   cases for the new algorithms to the KEX test.
2. Reorder the cannedKexAlgoNames with the ML-KEM algos first.
3. Add the new algos to wolfSSH_GetText().
4. Add comments and whitespace cleanup.
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ejohnstown ejohnstown assigned wolfSSL-Bot and unassigned ejohnstown Jan 14, 2026
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.

4 participants