refine deps

Cargo.toml

@@ -12,14 +12,14 @@ publish = false
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
 [features]
-default = ["h3"]
+default = ["http3"]
-h3 = ["quinn"]
+http3 = ["quinn", "h3", "h3-quinn"]
 
 [dependencies]
+env_logger = "0.9.0"
 anyhow = "1.0.58"
 clap = { version = "3.2.13", features = ["std", "cargo", "wrap_help"] }
-env_logger = "0.9.0"
+futures = { version = "0.3.21", features = ["alloc", "async-await"] }
-futures = "0.3.21"
 hyper = { version = "0.14.20", default-features = false, features = [
   "server",
   "http1",
@@ -36,26 +36,28 @@ tokio = { version = "1.20.0", default-features = false, features = [
   "macros",
 ] }
 tokio-rustls = { version = "0.23.4", features = ["early-data"] }
-rustls-pemfile = { version = "1.0.0" }
+rustls-pemfile = "1.0.0"
-rustls = "0.20.6"
+rustls = { version = "0.20.6", default-features = false }
 rand = "0.8.5"
-toml = "0.5.9"
+toml = { version = "0.5.9", default-features = false }
 rustc-hash = "1.1.0"
-serde = { version = "1.0.139", features = ["derive"] }
+serde = { version = "1.0.140", default-features = false, features = ["derive"] }
 hyper-rustls = { version = "0.23.0", default-features = false, features = [
   "tokio-runtime",
   "webpki-tokio",
   "http1",
   "http2",
 ] }
-quinn = { version = "0.8.3", optional = true }
-h3 = { git = "https://github.com/hyperium/h3.git" }
-h3-quinn = { git = "https://github.com/hyperium/h3.git" }
 bytes = "1.2.0"
+quinn = { version = "0.8.3", optional = true }
+h3 = { git = "https://github.com/hyperium/h3.git", optional = true }
+h3-quinn = { git = "https://github.com/hyperium/h3.git", optional = true }
+
 
 [target.'cfg(not(target_env = "msvc"))'.dependencies]
 tikv-jemallocator = "0.5.0"
 
+
 [dev-dependencies]
 
 

src/backend.rs

@@ -236,16 +236,16 @@ impl Backends {
       .with_no_client_auth()
       .with_cert_resolver(Arc::new(resolver));
 
-    #[cfg(feature = "h3")]
+    #[cfg(feature = "http3")]
     {
       server_config.alpn_protocols = vec![
         b"h3".to_vec(),
-        b"hq-29".to_vec(), // quinn draft example TODO: remove later
+        b"hq-29".to_vec(), // TODO: remove later?
         b"h2".to_vec(),
         b"http/1.1".to_vec(),
       ];
     }
-    #[cfg(not(feature = "h3"))]
+    #[cfg(not(feature = "http3"))]
     {
       server_config.alpn_protocols = vec![b"h2".to_vec(), b"http/1.1".to_vec()];
     }

src/config/parse.rs

@@ -157,6 +157,8 @@ pub fn parse_opts(globals: &mut Globals) -> Result<()> {
 
   // experimental
   if let Some(exp) = config.experimental {
+    #[cfg(feature = "http3")]
+    {
       if let Some(h3option) = exp.h3 {
         globals.http3 = true;
         info!("Experimental HTTP/3.0 is enabled. Note it is still very unstable.");
@@ -176,6 +178,8 @@ pub fn parse_opts(globals: &mut Globals) -> Result<()> {
           globals.h3_max_concurrent_unistream = x.into();
         }
       }
+    }
+
     if let Some(b) = exp.ignore_sni_consistency {
       globals.sni_consistency = !b;
       if b {

src/config/toml.rs

@@ -15,6 +15,7 @@ pub struct ConfigToml {
   pub experimental: Option<Experimental>,
 }
 
+#[cfg(feature = "http3")]
 #[derive(Deserialize, Debug, Default)]
 pub struct Http3Option {
   pub alt_svc_max_age: Option<u32>,
@@ -26,6 +27,7 @@ pub struct Http3Option {
 
 #[derive(Deserialize, Debug, Default)]
 pub struct Experimental {
+  #[cfg(feature = "http3")]
   pub h3: Option<Http3Option>,
   pub ignore_sni_consistency: Option<bool>,
 }

src/constants.rs

@@ -9,13 +9,13 @@ pub const MAX_CONCURRENT_STREAMS: u32 = 64;
 // #[cfg(feature = "tls")]
 pub const CERTS_WATCH_DELAY_SECS: u32 = 30;
 
-// #[cfg(feature = "h3")]
+// #[cfg(feature = "http3")]
 // pub const H3_RESPONSE_BUF_SIZE: usize = 65_536; // 64KB
-// #[cfg(feature = "h3")]
+// #[cfg(feature = "http3")]
 // pub const H3_REQUEST_BUF_SIZE: usize = 65_536; // 64KB // handled by quinn
 
 #[allow(non_snake_case)]
-#[cfg(feature = "h3")]
+#[cfg(feature = "http3")]
 pub mod H3 {
   pub const ALT_SVC_MAX_AGE: u32 = 3600;
   pub const REQUEST_MAX_BODY_SIZE: usize = 268_435_456; // 256MB

src/globals.rs

@@ -24,11 +24,18 @@ pub struct Globals {
 
   // experimentals
   pub sni_consistency: bool,
+
+  #[cfg(feature = "http3")]
   pub http3: bool,
+  #[cfg(feature = "http3")]
   pub h3_alt_svc_max_age: u32,
+  #[cfg(feature = "http3")]
   pub h3_request_max_body_size: usize,
+  #[cfg(feature = "http3")]
   pub h3_max_concurrent_bidistream: quinn::VarInt,
+  #[cfg(feature = "http3")]
   pub h3_max_concurrent_unistream: quinn::VarInt,
+  #[cfg(feature = "http3")]
   pub h3_max_concurrent_connections: u32,
 }
 

src/main.rs

@@ -77,11 +77,18 @@ fn main() {
       },
 
       sni_consistency: true,
+
+      #[cfg(feature = "http3")]
       http3: false,
+      #[cfg(feature = "http3")]
       h3_alt_svc_max_age: H3::ALT_SVC_MAX_AGE,
+      #[cfg(feature = "http3")]
       h3_request_max_body_size: H3::REQUEST_MAX_BODY_SIZE,
+      #[cfg(feature = "http3")]
       h3_max_concurrent_connections: H3::MAX_CONCURRENT_CONNECTIONS,
+      #[cfg(feature = "http3")]
       h3_max_concurrent_bidistream: H3::MAX_CONCURRENT_BIDISTREAM.into(),
+      #[cfg(feature = "http3")]
       h3_max_concurrent_unistream: H3::MAX_CONCURRENT_UNISTREAM.into(),
     };
 

src/msg_handler/handler.rs

@@ -200,7 +200,7 @@ where
     remove_hop_header(headers);
     add_header_entry_overwrite_if_exist(headers, "server", env!("CARGO_PKG_NAME"))?;
 
-    #[cfg(feature = "h3")]
+    #[cfg(feature = "http3")]
     {
       if self.globals.http3 {
         if let Some(port) = self.globals.https_port {

src/proxy/mod.rs

@@ -1,4 +1,4 @@
-#[cfg(feature = "h3")]
+#[cfg(feature = "http3")]
 mod proxy_h3;
 mod proxy_main;
 mod proxy_tls;

src/proxy/proxy_h3.rs

@@ -10,17 +10,7 @@ impl<T> Proxy<T>
 where
   T: Connect + Clone + Sync + Send + 'static,
 {
-  pub(super) fn connection_serve_h3(&self, conn: quinn::Connecting, tls_server_name: &[u8]) {
+  pub(super) async fn connection_serve_h3(self, conn: quinn::Connecting, tls_server_name: ServerNameLC) -> Result<()> {
-    let fut = self.clone().handle_connection_h3(conn, tls_server_name.to_vec());
-    self.globals.runtime_handle.spawn(async move {
-      // Timeout is based on underlying quic
-      if let Err(e) = fut.await {
-        warn!("QUIC or HTTP/3 connection failed: {}", e)
-      }
-    });
-  }
-
-  async fn handle_connection_h3(self, conn: quinn::Connecting, tls_server_name: ServerNameLC) -> Result<()> {
     let client_addr = conn.remote_address();
 
     match conn.await {

src/proxy/proxy_tls.rs

@@ -1,7 +1,5 @@
 use super::proxy_main::{LocalExecutor, Proxy};
 use crate::{constants::*, error::*, log::*};
-#[cfg(feature = "h3")]
-use futures::StreamExt;
 use futures::{future::FutureExt, select};
 use hyper::{client::connect::Connect, server::conn::Http};
 use rustls::ServerConfig;
@@ -13,6 +11,11 @@ use tokio::{
 };
 use tokio_rustls::TlsAcceptor;
 
+#[cfg(feature = "http3")]
+use futures::StreamExt;
+#[cfg(feature = "http3")]
+use quinn::{crypto::rustls::HandshakeData, Endpoint, ServerConfig as QuicServerConfig, TransportConfig};
+
 impl<T> Proxy<T>
 where
   T: Connect + Clone + Sync + Send + 'static,
@@ -76,9 +79,9 @@ where
     Ok(()) as Result<()>
   }
 
-  #[cfg(feature = "h3")]
+  #[cfg(feature = "http3")]
   async fn listener_service_h3(&self, mut server_crypto_rx: watch::Receiver<Option<Arc<ServerConfig>>>) -> Result<()> {
-    let mut transport_config_quic = quinn::TransportConfig::default();
+    let mut transport_config_quic = TransportConfig::default();
     transport_config_quic
       .max_concurrent_bidi_streams(self.globals.h3_max_concurrent_bidistream)
       .max_concurrent_uni_streams(self.globals.h3_max_concurrent_unistream);
@@ -89,10 +92,10 @@ where
       .generate_server_crypto_with_cert_resolver()
       .await?;
 
-    let mut server_config_h3 = quinn::ServerConfig::with_crypto(Arc::new(server_crypto));
+    let mut server_config_h3 = QuicServerConfig::with_crypto(Arc::new(server_crypto));
     server_config_h3.transport = Arc::new(transport_config_quic);
     server_config_h3.concurrent_connections(self.globals.h3_max_concurrent_connections);
-    let (endpoint, mut incoming) = quinn::Endpoint::server(server_config_h3, self.listening_on)?;
+    let (endpoint, mut incoming) = Endpoint::server(server_config_h3, self.listening_on)?;
     info!("Start UDP proxy serving with HTTP/3 request for configured host names");
 
     let mut server_crypto: Option<Arc<ServerConfig>> = None;
@@ -103,15 +106,14 @@ where
             continue;
           }
           let mut conn = new_conn.unwrap();
-          let hsd = if let Ok(h) = conn.handshake_data().await {
+          let hsd = match conn.handshake_data().await {
-            h
+            Ok(h) => h,
-          } else {
+            Err(_) => continue
-            continue
           };
-          let hsd_downcast = if let Ok(d) = hsd.downcast::<quinn::crypto::rustls::HandshakeData>() {
+
-            d
+          let hsd_downcast = match hsd.downcast::<HandshakeData>() {
-          } else {
+            Ok(d) => d,
-            continue;
+            Err(_) => continue
           };
           let new_server_name = if let Some(sn) = hsd_downcast.server_name {
             sn.as_bytes().to_ascii_lowercase()
@@ -125,7 +127,13 @@ where
           );
           // TODO: server_nameをここで出してどんどん深く投げていくのは効率が悪い。connecting -> connectionsの後でいいのでは？
           // TODO: 通常のTLSと同じenumか何かにまとめたい
-          self.clone().connection_serve_h3(conn, new_server_name.as_ref());
+          let fut = self.clone().connection_serve_h3(conn, new_server_name);
+          self.globals.runtime_handle.spawn(async move {
+            // Timeout is based on underlying quic
+            if let Err(e) = fut.await {
+              warn!("QUIC or HTTP/3 connection failed: {}", e)
+            }
+          });
         }
         _ = server_crypto_rx.changed().fuse() => {
           if server_crypto_rx.borrow().is_none() {
@@ -134,7 +142,7 @@ where
           server_crypto = server_crypto_rx.borrow().clone();
           if server_crypto.is_some(){
             debug!("Reload server crypto");
-            endpoint.set_server_config(Some(quinn::ServerConfig::with_crypto(server_crypto.clone().unwrap())));
+            endpoint.set_server_config(Some(QuicServerConfig::with_crypto(server_crypto.clone().unwrap())));
           }
         }
         complete => break
@@ -146,7 +154,7 @@ where
 
   pub async fn start_with_tls(self, server: Http<LocalExecutor>) -> Result<()> {
     let (tx, rx) = watch::channel::<Option<Arc<ServerConfig>>>(None);
-    #[cfg(not(feature = "h3"))]
+    #[cfg(not(feature = "http3"))]
     {
       select! {
         _= self.cert_service(tx).fuse() => {
@@ -162,7 +170,7 @@ where
       };
       Ok(())
     }
-    #[cfg(feature = "h3")]
+    #[cfg(feature = "http3")]
     {
       if self.globals.http3 {
         select! {