From 9107e6f3ebbe65a8d76e899693c1f6e66531d569 Mon Sep 17 00:00:00 2001
From: Paul Campbell <pcampbell@kemitix.net>
Date: Wed, 8 May 2019 13:00:03 +0100
Subject: [PATCH] [Reactives3client] use for-comprehension

---
 src/main/scala/net/kemitix/s3thorp/ReactiveS3Client.scala | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/main/scala/net/kemitix/s3thorp/ReactiveS3Client.scala b/src/main/scala/net/kemitix/s3thorp/ReactiveS3Client.scala
index 3e282e9..a8cd732 100644
--- a/src/main/scala/net/kemitix/s3thorp/ReactiveS3Client.scala
+++ b/src/main/scala/net/kemitix/s3thorp/ReactiveS3Client.scala
@@ -14,7 +14,9 @@ class ReactiveS3Client extends S3Client {
       .bucket(bucket)
       .key(key)
       .build()
-    s3Client.headObject(request).
-      map(r => (r.eTag(), r.lastModified()))
+    for {
+      response <- s3Client.headObject(request)
+      // TODO catch 404 error when key doesn't exist
+    } yield (response.eTag(), response.lastModified())
   }
 }