Add IoError to TransportError

Author: AlfioEmanueleFresta

libwebauthn/src/transport/ble/channel.rs

@@ -7,9 +7,7 @@ use crate::proto::ctap1::apdu::{ApduRequest, ApduResponse};
 use crate::proto::ctap2::cbor::{CborRequest, CborResponse};
 use crate::proto::CtapError;
 use crate::transport::ble::btleplug;
-use crate::transport::channel::{
-    AuthTokenData, Channel, ChannelStatus, Ctap2AuthTokenStore,
-};
+use crate::transport::channel::{AuthTokenData, Channel, ChannelStatus, Ctap2AuthTokenStore};
 use crate::transport::device::SupportedProtocols;
 use crate::transport::error::{Error, TransportError};
 use crate::UxUpdate;
@@ -129,7 +127,9 @@ impl<'a> Channel for BleChannel<'a> {
         debug!("Sending CBOR request");
         trace!(?request);
 
-        let cbor_request = request.raw_long().or(Err(TransportError::InvalidFraming))?;
+        let cbor_request = request
+            .raw_long()
+            .map_err(|e| TransportError::IoError(e.kind()))?;
         let request_frame = BleFrame::new(BleCommand::Msg, &cbor_request);
         self.connection
             .frame_send(&request_frame)

libwebauthn/src/transport/cable/tunnel.rs

@@ -453,7 +453,9 @@ async fn connection_send(
     debug!("Sending CBOR request");
     trace!(?request);
 
-    let cbor_request = request.raw_long().or(Err(TransportError::InvalidFraming))?;
+    let cbor_request = request
+        .raw_long()
+        .map_err(|e| TransportError::IoError(e.kind()))?;
     if cbor_request.len() > MAX_CBOR_SIZE {
         error!(
             cbor_request_len = cbor_request.len(),

libwebauthn/src/transport/error.rs

@@ -40,6 +40,8 @@ pub enum TransportError {
     InvalidKey,
     #[error("invalid signature")]
     InvalidSignature,
+    #[error("input/output error: {0}")]
+    IoError(std::io::ErrorKind),
 }
 
 #[derive(thiserror::Error, Debug, Copy, Clone, PartialEq)]

libwebauthn/src/transport/hid/channel.rs

@@ -448,10 +448,9 @@ impl Channel for HidChannel<'_> {
         let cid = self.init.cid;
         debug!({ cid }, "Sending APDU request");
         trace!(?request);
-        let Ok(apdu_raw) = request.raw_long() else {
-            warn!("Failed to serialize APDU request");
-            return Err(Error::Transport(TransportError::InvalidFraming));
-        };
+        let apdu_raw = request
+            .raw_long()
+            .map_err(|e| TransportError::IoError(e.kind()))?;
         self.hid_send(&HidMessage::new(cid, HidCommand::Msg, &apdu_raw))
             .await?;
         Ok(())