Fixed Renderer to handle properly Manifest and Blob responses.

closes #7620 https://pulp.plan.io/issues/7620
pulp · Nov 3, 2020 · 970e2cf · 970e2cf
1 parent c01a669
commit 970e2cf
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/CHANGES/7620.bugfix b/CHANGES/7620.bugfix
@@ -0,0 +1 @@
+Fixed Renderer to handle properly Manifest and Blob responses.
diff --git a/pulp_container/app/registry_api.py b/pulp_container/app/registry_api.py
@@ -108,9 +108,9 @@ def __init__(self, reference):
         )
 
 
-class ManifestRenderer(BaseRenderer):
+class ContentRenderer(BaseRenderer):
     """
-    Rendered class for rendering Manifest responses.
+    Rendered class for rendering Manifest and Blob responses.
     """
 
     media_type = "*/*"
@@ -504,6 +504,8 @@ class Blobs(RedirectsMixin, ContainerRegistryApiMixin, ViewSet):
     ViewSet for interacting with Blobs
     """
 
+    renderer_classes = [ContentRenderer]
+
     def head(self, request, path, pk=None):
         """
         Responds to HEAD requests about blobs
@@ -529,10 +531,10 @@ def get(self, request, path, pk=None):
 
 class Manifests(RedirectsMixin, ContainerRegistryApiMixin, ViewSet):
     """
-    ViewSet for intereacting with Manifests
+    ViewSet for interacting with Manifests
     """
 
-    renderer_classes = [ManifestRenderer]
+    renderer_classes = [ContentRenderer]
     # The lookup regex does not allow /, ^, &, *, %, !, ~, @, #, +, =, ?
     lookup_value_regex = "[^/^&*%!~@#+=?]+"
 

diff --git a/pulp_container/app/tasks/sync_stages.py b/pulp_container/app/tasks/sync_stages.py
@@ -330,7 +330,6 @@ def create_blob(self, man_dc, blob_data):
             url=blob_url,
             relative_path=digest,
             remote=self.remote,
-            extra_data={"headers": V2_ACCEPT_HEADERS},
             deferred_download=self.deferred_download,
         )
         blob_dc = DeclarativeContent(content=blob, d_artifacts=[da])