Serializer.validate must return validated data. Fixes #1441 (#1432)

(cherry picked from commit 69e2017)
pulp · May 9, 2023 · ac08e5d · ac08e5d
1 parent 4f75527
commit ac08e5d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/CHANGES/1441.bugfix b/CHANGES/1441.bugfix
@@ -0,0 +1 @@
+Fix traceback when publishing a collection to the v2 API endpoint
diff --git a/pulp_ansible/app/galaxy/serializers.py b/pulp_ansible/app/galaxy/serializers.py
@@ -191,7 +191,9 @@ class GalaxyCollectionUploadSerializer(serializers.Serializer):
 
     def validate(self, data):
         """Ensure duplicate artifact isn't uploaded."""
+        data = super().validate(data)
         sha256 = data["file"].hashers["sha256"].hexdigest()
         artifact = Artifact.objects.filter(sha256=sha256).first()
         if artifact:
             raise serializers.ValidationError(_("Artifact already exists"))
+        return data