Add context to error message and update test

TBD54566975 · Sep 23, 2024 · 6fc91ec · 6fc91ec
1 parent feb9760
commit 6fc91ec
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 9 deletions.
diff --git a/crates/web5/src/credentials/create.rs b/crates/web5/src/credentials/create.rs
@@ -572,9 +572,10 @@ mod tests {
         match result {
             Err(Web5Error::Network(err_msg)) => {
                 assert!(
-                    err_msg.contains("failed to lookup address information"),
-                    "Error message is: {}", err_msg
-                )            
+                    err_msg.contains("failed to connect to host"),
+                    "Error message is: {}",
+                    err_msg
+                )
             }
             _ => panic!(
                 "expected Web5Error::Network with specific message but got {:?}",

diff --git a/crates/web5/src/credentials/verifiable_credential_1_1.rs b/crates/web5/src/credentials/verifiable_credential_1_1.rs
@@ -798,8 +798,9 @@ mod tests {
             match result {
                 Err(Web5Error::Network(err_msg)) => {
                     assert!(
-                        err_msg.contains("failed to lookup address information"),
-                        "Error message is: {}", err_msg
+                        err_msg.contains("failed to connect to host"),
+                        "Error message is: {}",
+                        err_msg
                     )
                 }
                 _ => panic!(

diff --git a/crates/web5/src/http.rs b/crates/web5/src/http.rs
@@ -67,8 +67,10 @@ fn transmit(destination: &Destination, request: &[u8]) -> Result<Vec<u8>> {
         let rc_config = Arc::new(config); // Arc allows sharing the config
 
         // Make the TCP connection to the server
-        let stream = TcpStream::connect((&destination.host[..], destination.port))
-            .map_err(|err| Web5Error::Network(err.to_string()))?;
+        let stream =
+            TcpStream::connect((&destination.host[..], destination.port)).map_err(|err| {
+                Web5Error::Network(format!("failed to connect to host: {}", err))
+            })?;
 
         // Convert the server name to the expected type for TLS validation
         let server_name = ServerName::try_from(destination.host.clone())
@@ -90,8 +92,10 @@ fn transmit(destination: &Destination, request: &[u8]) -> Result<Vec<u8>> {
             .map_err(|err| Web5Error::Network(err.to_string()))?;
     } else {
         // HTTP connection
-        let mut stream = TcpStream::connect((&destination.host[..], destination.port))
-            .map_err(|err| Web5Error::Network(err.to_string()))?;
+        let mut stream =
+            TcpStream::connect((&destination.host[..], destination.port)).map_err(|err| {
+                Web5Error::Network(format!("failed to connect to host: {}", err))
+            })?;
 
         stream
             .write_all(request)