refactor: Include more details in errors

Author: LeanderBB

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "proton-api-rs"
 authors = ["Leander Beernaert <lbb-dev@pm.me>"]
-version = "0.5.0"
+version = "0.5.1"
 edition = "2021"
 license = "AGPL-3.0-only"
 description = "Unofficial implemention of proton REST API in rust"

src/client/error.rs

@@ -17,16 +17,16 @@ pub enum RequestError {
 #[derive(Debug, Error)]
 /// Errors that may occur during an HTTP request, mostly related to network.
 pub enum HttpClientError {
-    #[error("A redirect error occurred at '{0}")]
-    Redirect(String),
+    #[error("A redirect error occurred at '{0}: {1}")]
+    Redirect(String, #[source] anyhow::Error),
     #[error("Connection timed out")]
-    Timeout,
-    #[error("Connection error occurred")]
-    Connection,
-    #[error("An error occurred related to either the request or response body")]
-    Body,
-    #[error("An error occurred preparing the request")]
-    Request,
+    Timeout(#[source] anyhow::Error),
+    #[error("Connection error: {0}")]
+    Connection(#[source] anyhow::Error),
+    #[error("Request/Response body error: {0}")]
+    Body(#[source] anyhow::Error),
+    #[error("Request error:{0}")]
+    Request(#[source] anyhow::Error),
     #[error("Unexpected error occurred: {0}")]
     Other(#[source] anyhow::Error),
 }

src/client/http_client/reqwest_client.rs

@@ -167,22 +167,23 @@ impl From<reqwest::Error> for HttpClientError {
     fn from(value: reqwest::Error) -> Self {
         #[cfg(not(target_arch = "wasm32"))]
         if value.is_connect() {
-            return HttpClientError::Connection;
+            return HttpClientError::Connection(anyhow::Error::new(value));
         }
 
         if value.is_body() {
-            HttpClientError::Body
+            HttpClientError::Body(anyhow::Error::new(value))
         } else if value.is_redirect() {
             HttpClientError::Redirect(
                 value
                     .url()
                     .map(|v| v.to_string())
                     .unwrap_or("Unknown URL".to_string()),
+                anyhow::Error::new(value),
             )
         } else if value.is_timeout() {
-            HttpClientError::Timeout
+            HttpClientError::Timeout(anyhow::Error::new(value))
         } else if value.is_request() {
-            HttpClientError::Request
+            HttpClientError::Request(anyhow::Error::new(value))
         } else {
             HttpClientError::Other(anyhow::Error::new(value))
         }