Merge pull request #696 from RockefellerArchiveCenter/issue-695

Truncate machine name before creating bucket
RockefellerArchiveCenter · Dec 5, 2024 · c91d631 · c91d631
2 parents 7d3a4c5 + 37bdbf1
commit c91d631
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/bag_transfer/models.py b/bag_transfer/models.py
@@ -64,7 +64,9 @@ def admin_users(self):
     @property
     def upload_target(self):
         if settings.S3_USE:
-            return f"{settings.S3_PREFIX}-{self.machine_name}-upload"
+            max_len = 63 - len(settings.S3_PREFIX) - 8  # ensure bucket name will not exceed max length
+            machine_name_truncated = self.machine_name[:max_len]
+            return f"{settings.S3_PREFIX}-{machine_name_truncated}-upload"
         else:
             return join(settings.TRANSFER_UPLOADS_ROOT.rstrip("/"), self.machine_name, "upload")