chore: box errors

src/auth.rs

@@ -25,7 +25,7 @@ impl fmt::Debug for Auth {
                 .field("password", &"******")
                 .finish(),
 
-            Auth::Jwt(token) => f.debug_struct("JwtAuth").field("token", token).finish(),
+            Auth::Jwt(_) => f.debug_struct("JwtAuth").field("token", &"******").finish(),
         }
     }
 }

src/client.rs

@@ -247,7 +247,6 @@ impl ClientBuilder {
         self
     }
 
-    #[allow(clippy::result_large_err)]
     pub fn build(self) -> Result<Client> {
         let session = self.session.build()?;
         let max_attempt = self.max_attempt;
@@ -665,7 +664,6 @@ impl Client {
     }
 
     #[cfg(feature = "spooling")]
-    #[allow(clippy::result_large_err)]
     fn decode_segments<T: Trino + 'static>(
         &self,
         encoding: &str,

src/error.rs

@@ -6,7 +6,6 @@ use thiserror::Error;
 use crate::models::QueryError;
 
 #[derive(Error, Debug)]
-#[allow(clippy::result_large_err)]
 pub enum Error {
     #[error("invalid catalog")]
     InvalidCatalog,
@@ -39,7 +38,7 @@ pub enum Error {
     #[error("basic auth can not be used with http")]
     BasicAuthWithHttp,
     #[error("http error, reason: {0}")]
-    HttpError(#[from] reqwest::Error),
+    HttpError(#[source] Box<reqwest::Error>),
     #[error("http not ok, code: {0}, reason: {1}")]
     HttpNotOk(StatusCode, String),
     #[error("query error, reason: {0}")]
@@ -56,6 +55,12 @@ pub enum Error {
     InternalError(String),
 }
 
+impl From<reqwest::Error> for Error {
+    fn from(err: reqwest::Error) -> Self {
+        Error::HttpError(Box::new(err))
+    }
+}
+
 impl From<QueryError> for Error {
     fn from(err: QueryError) -> Self {
         Error::QueryError(Box::new(err))

src/session.rs

@@ -92,7 +92,6 @@ impl SessionBuilder {
         }
     }
 
-    #[allow(clippy::result_large_err)]
     pub fn build(self) -> Result<Session> {
         let scheme = if self.secure {
             Scheme::HTTPS

src/ssl.rs

@@ -17,7 +17,6 @@ impl Default for Ssl {
 }
 
 impl Ssl {
-    #[allow(clippy::result_large_err)]
     pub fn read_pem<P: AsRef<Path>>(root_certificate_path: &P) -> Result<Certificate> {
         let buf = Self::read_file(&root_certificate_path)?;
         match reqwest::Certificate::from_pem(&buf) {
@@ -29,7 +28,6 @@ impl Ssl {
         }
     }
 
-    #[allow(clippy::result_large_err)]
     pub fn read_der<P: AsRef<Path>>(root_certificate_path: &P) -> Result<Certificate> {
         let buf = Self::read_file(&root_certificate_path)?;
         match reqwest::Certificate::from_der(&buf) {
@@ -41,7 +39,6 @@ impl Ssl {
         }
     }
 
-    #[allow(clippy::result_large_err)]
     fn read_file<P: AsRef<Path>>(file_path: &P) -> Result<Vec<u8>> {
         let mut buf = Vec::new();
         std::fs::File::open(file_path)