RTG-3333 Support X25519MLKEM768 by default, but don't sent it as client
X25519MLKEM768 is the standardised successor of the preliminary X25519Kyber768Draft00. Latest browsers have switched to X25519MLKEM768. Cloudflare supports both on the edge. We've had support for X25519MLKEM768 in this crate for a while, but didn't enable by default. We're now enabling serverside support by default. We also let clients advertise support when set to kx-client-pq-supported. We don't enable support by default yet for clients set to kx-client-pq-preferred, as that would cause an extra round-trip due to HelloRetryRequest if the server doesn't support X25519MLKEM768 yet. BoringSSL against which we build must support X25519MLKEM768, otherwise this will fail.
This commit is contained in:
parent
796afe1637
commit
6ca27a7738
|
|
@ -2748,7 +2748,7 @@ impl SslRef {
|
||||||
if cfg!(feature = "kx-client-nist-required") {
|
if cfg!(feature = "kx-client-nist-required") {
|
||||||
"P-256:P-384:P-521:P256Kyber768Draft00"
|
"P-256:P-384:P-521:P256Kyber768Draft00"
|
||||||
} else {
|
} else {
|
||||||
"X25519:P-256:P-384:P-521:X25519Kyber768Draft00:P256Kyber768Draft00"
|
"X25519:P-256:P-384:P-521:X25519MLKEM768:X25519Kyber768Draft00:P256Kyber768Draft00"
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
if cfg!(feature = "kx-client-nist-required") {
|
if cfg!(feature = "kx-client-nist-required") {
|
||||||
|
|
@ -2764,8 +2764,10 @@ impl SslRef {
|
||||||
|
|
||||||
#[cfg(feature = "kx-safe-default")]
|
#[cfg(feature = "kx-safe-default")]
|
||||||
fn server_set_default_curves_list(&mut self) {
|
fn server_set_default_curves_list(&mut self) {
|
||||||
self.set_curves_list("X25519Kyber768Draft00:P256Kyber768Draft00:X25519:P-256:P-384")
|
self.set_curves_list(
|
||||||
.expect("invalid default server curves list");
|
"X25519MLKEM768:X25519Kyber768Draft00:P256Kyber768Draft00:X25519:P-256:P-384",
|
||||||
|
)
|
||||||
|
.expect("invalid default server curves list");
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Returns the [`SslCurve`] used for this `SslRef`.
|
/// Returns the [`SslCurve`] used for this `SslRef`.
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue