Add error logs to S3 implementation

evervault · Aug 31, 2023 · 166d525 · 166d525
1 parent 0a9bed9
commit 166d525
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "storage-client-interface"
-version = "0.1.0"
+version = "0.2.0"
 edition = "2021"
 license = "Apache-2.0"
 description = "A Rust library for exposing the StorageClientInterface trait for interacting with a storage backend. Used by Evervault Cages."

diff --git a/src/s3.rs b/src/s3.rs
@@ -67,6 +67,7 @@ impl StorageClientInterface for StorageClient {
             Err(err) => match err.into_service_error() {
                 GetObjectError::NoSuchKey(_) => return Ok(None),
                 err => {
+                    println!("Error getting object from S3: {:?}", err);
                     return Err(StorageClientError::GetObject(err.to_string()));
                 }
             },
@@ -96,7 +97,10 @@ impl StorageClientInterface for StorageClient {
             .body(ByteStream::from(body_bytes))
             .send()
             .await
-            .map_err(ClientError::PutObject)?;
+            .map_err(|err| {
+                println!("Error puttin object to S3: {:?}", err);
+                ClientError::PutObject(err)
+            })?;
 
         Ok(())
     }
@@ -109,7 +113,10 @@ impl StorageClientInterface for StorageClient {
             .key(key)
             .send()
             .await
-            .map_err(ClientError::DeleteObject)?;
+            .map_err(|err| {
+                println!("Error deleting object in S3: {:?}", err);
+                ClientError::DeleteObject(err)
+            })?;
 
         Ok(())
     }