Use custom filename on download only if backup file name is backup slug

home-assistant · Jan 22, 2025 · bcb8f2a · bcb8f2a
1 parent 8c14810
commit bcb8f2a
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/supervisor/api/backups.py b/supervisor/api/backups.py
@@ -460,10 +460,14 @@ async def download(self, request: web.Request):
             raise APIError(f"Backup {backup.slug} is not in location {location}")
 
         _LOGGER.info("Downloading backup %s", backup.slug)
-        response = web.FileResponse(backup.all_locations[location])
+        filename = backup.all_locations[location]
+        response = web.FileResponse(filename)
         response.content_type = CONTENT_TYPE_TAR
+
+        if filename == f"{backup.slug}.tar":
+            filename = f"{RE_SLUGIFY_NAME.sub('_', backup.name)}.tar"
         response.headers[CONTENT_DISPOSITION] = (
-            f"attachment; filename={RE_SLUGIFY_NAME.sub('_', backup.name)}.tar"
+            f"attachment; filename={filename}"
         )
         return response