Add an option to retrieve TLS errors

Signed-off-by: Mateusz Szczygieł <[email protected]>

Author: matszczygiel

quinn-proto/src/connection/mod.rs

@@ -2605,6 +2605,7 @@ impl Connection {
                                 code: TransportErrorCode::crypto(0x6d),
                                 frame: None,
                                 reason: "transport parameters missing".into(),
+                                tls: None,
                             })?;
 
                     if self.has_0rtt() {
@@ -2678,6 +2679,7 @@ impl Connection {
                                 code: TransportErrorCode::crypto(0x6d),
                                 frame: None,
                                 reason: "transport parameters missing".into(),
+                                tls: None,
                             })?;
                     self.handle_peer_params(params)?;
                     self.issue_first_cids(now);

quinn-proto/src/crypto/rustls.rs

@@ -110,6 +110,7 @@ impl crypto::Session for TlsSession {
                     code: TransportErrorCode::crypto(alert.into()),
                     frame: None,
                     reason: e.to_string(),
+                    tls: Some(e),
                 }
             } else {
                 TransportError::PROTOCOL_VIOLATION(format!("TLS error: {e}"))

quinn-proto/src/transport_error.rs

@@ -8,16 +8,26 @@ use crate::{
 };
 
 /// Transport-level errors occur when a peer violates the protocol specification
-#[derive(Debug, Clone, Eq, PartialEq)]
+#[derive(Debug, Clone)]
 pub struct Error {
     /// Type of error
     pub code: Code,
     /// Frame type that triggered the error
     pub frame: Option<frame::FrameType>,
     /// Human-readable explanation of the reason
     pub reason: String,
+    /// An underlying TLS layer error
+    pub tls: Option<rustls::Error>,
 }
 
+impl PartialEq for Error {
+    fn eq(&self, other: &Self) -> bool {
+        self.code == other.code && self.frame == other.frame && self.reason == other.reason
+    }
+}
+
+impl Eq for Error {}
+
 impl fmt::Display for Error {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         self.code.fmt(f)?;
@@ -39,6 +49,7 @@ impl From<Code> for Error {
             code: x,
             frame: None,
             reason: "".to_string(),
+            tls: None,
         }
     }
 }
@@ -79,6 +90,7 @@ macro_rules! errors {
                     code: Code::$name,
                     frame: None,
                     reason: reason.into(),
+                    tls: None,
                 }
             }
             )*

quinn/src/connection.rs

@@ -1078,6 +1078,7 @@ impl State {
                         code: proto::TransportErrorCode::INTERNAL_ERROR,
                         frame: None,
                         reason: "endpoint driver future was dropped".to_string(),
+                        tls: None,
                     }));
                 }
                 Poll::Pending => {