fix: replace get_object_acl with head_object (#90)

thumbor_aws/result_storage.py

@@ -214,7 +214,7 @@ async def last_updated(  # pylint: disable=invalid-overridden-method
         file_abspath = self.normalize_path(path)
         logger.debug("[RESULT_STORAGE] getting from %s", file_abspath)
 
-        response = await self.get_object_acl(file_abspath)
+        response = await self.get_object_metadata(file_abspath)
         return datetime.strptime(
             response["ResponseMetadata"]["HTTPHeaders"]["last-modified"],
             "%a, %d %b %Y %H:%M:%S %Z",

thumbor_aws/s3_client.py

@@ -173,18 +173,23 @@ async def object_exists(self, filepath: str):
 
         async with self.get_client() as client:
             try:
-                await client.get_object_acl(
+                await client.head_object(
                     Bucket=self.bucket_name, Key=filepath
                 )
                 return True
             except client.exceptions.NoSuchKey:
                 return False
+            except client.exceptions.ClientError as err:
+                # NOTE: This case is required because of https://github.com/boto/boto3/issues/2442
+                if err.response["Error"]["Code"] == "404":
+                    return False
+                raise
 
-    async def get_object_acl(self, filepath: str):
+    async def get_object_metadata(self, filepath: str):
         """Gets an object's metadata"""
 
         async with self.get_client() as client:
-            return await client.get_object_acl(
+            return await client.head_object(
                 Bucket=self.bucket_name, Key=filepath
             )