Add comment for why we need to seek the download to the start

ministryofjustice · Aug 16, 2023 · 0f311be · 0f311be
1 parent 662d385
commit 0f311be
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/cla_backend/libs/aws/s3.py b/cla_backend/libs/aws/s3.py
@@ -26,6 +26,8 @@ def download_file(cls, bucket_name, key):
             obj = cls.get_s3_connection(bucket_name).bucket.Object(cls.clean_name(key))
             data = NamedTemporaryFile()
             obj.download_fileobj(data)
+            # This required otherwise any file reads will start at the end  which
+            # leads to an empty file being downloaded (zero bytes)
             data.seek(0)
             return {"headers": {"Content-Type": obj.content_type}, "body": data}
         except ClientError: