Content app now properly sets Content-Type header for S3

fixes: #9216
pulp · Aug 16, 2021 · feede7b · feede7b
1 parent d8fe016
commit feede7b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/CHANGES/9216.bugfix b/CHANGES/9216.bugfix
@@ -0,0 +1 @@
+Content app now properly sets Content-Type header for artifacts being served from S3
diff --git a/pulpcore/content/handler.py b/pulpcore/content/handler.py
@@ -788,6 +788,8 @@ async def _serve_content_artifact(self, content_artifact, headers):
         elif settings.DEFAULT_FILE_STORAGE == "storages.backends.s3boto3.S3Boto3Storage":
             content_disposition = f"attachment;filename={content_artifact.relative_path}"
             parameters = {"ResponseContentDisposition": content_disposition}
+            if headers.get("Content-Type"):
+                parameters["ResponseContentType"] = headers.get("Content-Type")
             url = URL(
                 artifact_file.storage.url(artifact_file.name, parameters=parameters), encoded=True
             )