minor fix

pagopa · Jun 10, 2024 · fb96169 · fb96169
1 parent f352cc9
commit fb96169
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/file-manager/src/main/scala/it/pagopa/interop/commons/files/service/impl/S3ManagerImpl.scala b/file-manager/src/main/scala/it/pagopa/interop/commons/files/service/impl/S3ManagerImpl.scala
@@ -185,7 +185,7 @@ final class S3ManagerImpl(blockingExecutionContext: ExecutionContextExecutor)(
 
       val presignRequest: GetObjectPresignRequest = GetObjectPresignRequest
         .builder()
-        .signatureDuration(Duration.ofMinutes(durationInMinutes.length))
+        .signatureDuration(Duration.ofMinutes(durationInMinutes.toMinutes))
         .getObjectRequest(objectRequest)
         .build()
 
@@ -211,7 +211,7 @@ final class S3ManagerImpl(blockingExecutionContext: ExecutionContextExecutor)(
 
       val presignRequest: PutObjectPresignRequest = PutObjectPresignRequest
         .builder()
-        .signatureDuration(Duration.ofMinutes(durationInMinutes.length))
+        .signatureDuration(Duration.ofMinutes(durationInMinutes.toMinutes))
         .putObjectRequest(objectRequest)
         .build()
 

diff --git a/file-manager/src/test/scala/it/pagopa/interop/commons/files/S3FileManagerTest.scala b/file-manager/src/test/scala/it/pagopa/interop/commons/files/S3FileManagerTest.scala
@@ -107,11 +107,11 @@ class S3FileManagerTest
       assert(contentMap == expected)
     }
     "Get presigned url" in {
-      val url = fileManager.generateGetPresignedUrl("testBucket", "testFolder", "testFile", 5).get
+      val url = fileManager.generateGetPresignedUrl("testBucket", "testFolder", "testFile", FiniteDuration(5, MINUTES)).get
       assert(url.nonEmpty)
     }
     "Put presigned url" in {
-      val url = fileManager.generatePutPresignedUrl("testBucket", "testFolder", "testFile", 1).get
+      val url = fileManager.generatePutPresignedUrl("testBucket", "testFolder", "testFile", FiniteDuration(1, MINUTES)).get
       assert(url.nonEmpty)
     }
   }