Add condition to ignore empty bucket

temba/archives/models.py

@@ -114,7 +114,7 @@ def get_download_link(self):
         if self.url:
             s3_client = s3.client()
             bucket, key = self.get_storage_location()
-            if bucket == "":
+            if not bucket:
                 return ""
             s3_params = {
                 "Bucket": bucket,
@@ -173,10 +173,12 @@ def iter_all_records(
 
         archives = cls._get_covering_period(org, archive_type, after, before)
 
+        # Random comment to verify if we are changing the image
         def generator():
             for archive in archives:
-                for record in archive.iter_records(where=where):
-                    yield record
+                if archive:
+                    for record in archive.iter_records(where=where):
+                        yield record
 
         return generator()
 
@@ -186,9 +188,12 @@ def iter_records(self, *, where: dict = None):
         """
 
         s3_client = s3.client()
+        bucket, key = self.get_storage_location()
+
+        if not bucket:
+            return
 
         if where:
-            bucket, key = self.get_storage_location()
             response = s3_client.select_object_content(
                 Bucket=bucket,
                 Key=key,
@@ -205,7 +210,6 @@ def generator():
             return generator()
 
         else:
-            bucket, key = self.get_storage_location()
             s3_obj = s3_client.get_object(Bucket=bucket, Key=key)
             return jsonlgz_iterate(s3_obj["Body"])