From c2b043e524a64296b8f79272e05d7f2da0600267 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marcel=20M=C3=A4rtens?= Date: Sun, 3 Jul 2022 20:51:49 +0200 Subject: [PATCH] add a comment about QUIC like Isse proposed --- network/src/channel.rs | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/network/src/channel.rs b/network/src/channel.rs index 592780d874..4fd31f1c21 100644 --- a/network/src/channel.rs +++ b/network/src/channel.rs @@ -309,6 +309,10 @@ impl Protocols { let metrics = ProtocolMetricCache::new(&cid.to_string(), Arc::clone(&metrics)); match Protocols::new_quic(connection, true, metrics).await { Ok(quic) => { + // TODO: we cannot guess the client hostname in quic server here. + // though we need it for the certificate to be validated, in the future + // this will either go away with new auth, or we have to do something like + // a reverse DNS lookup let connect_addr = ConnectAddr::Quic( addr, quinn::ClientConfig::with_native_roots(),