Add logging in s3 reactive storage where content id and length are set

paulcwarren · May 3, 2024 · dc647ab · dc647ab
1 parent 075256c
commit dc647ab
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/...c/main/java/internal/org/springframework/content/s3/store/DefaultReactiveS3StoreImpl.java b/...c/main/java/internal/org/springframework/content/s3/store/DefaultReactiveS3StoreImpl.java
@@ -123,9 +123,19 @@ public Mono<S> setContent(S entity, PropertyPath path, long contentLen, Flux<Byt
 
         return Mono.fromFuture(future)
           .map((response) -> {
-            property.setContentId(entity, s3ObjectId.getKey(), null);
-            property.setContentLength(entity, contentLen);
-            return entity;
+              try {
+                  property.setContentId(entity, s3ObjectId.getKey(), null);
+              } catch (Exception e) {
+                  logger.error("Error setting content id " + s3ObjectId.getKey(), e);
+                  throw e;
+              }
+              try {
+                  property.setContentLength(entity, contentLen);
+              } catch (Exception e) {
+                  logger.error("Error setting content length " + contentLen, e);
+                  throw e;
+              }
+              return entity;
           }
         );
     }