add a comment about QUIC like Isse proposed

This commit is contained in:
Marcel Märtens 2022-07-03 20:51:49 +02:00
parent b5d0ee22e4
commit c2b043e524

View File

@ -309,6 +309,10 @@ impl Protocols {
let metrics = ProtocolMetricCache::new(&cid.to_string(), Arc::clone(&metrics)); let metrics = ProtocolMetricCache::new(&cid.to_string(), Arc::clone(&metrics));
match Protocols::new_quic(connection, true, metrics).await { match Protocols::new_quic(connection, true, metrics).await {
Ok(quic) => { 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( let connect_addr = ConnectAddr::Quic(
addr, addr,
quinn::ClientConfig::with_native_roots(), quinn::ClientConfig::with_native_roots(),