feat: Add title in add_ingredient_file

if no title exists use the filename from the path.

Author: gpeacock

c2pa/c2pa_api/c2pa_api.py

@@ -114,6 +114,10 @@ def add_ingredient(self, ingredient, format, stream):
 
     def add_ingredient_file(self, ingredient, path):
         format = os.path.splitext(path)[1][1:]
+        if "title" not in ingredient:
+            if isinstance(ingredient, str):
+                ingredient = json.loads(ingredient)
+            ingredient["title"] = os.path.basename(path)
         with open(path, "rb") as file:
             return self.add_ingredient(ingredient, format, file)
 

tests/test_api.py

@@ -51,6 +51,14 @@ def getitem(d, key):
     ]
 }
 
+ingredient_def = {
+    "relationship": "parentOf",
+    "thumbnail": {
+        "identifier": "A.jpg",
+        "format": "image/jpeg"
+    }
+}
+
 def test_v2_read_cloud_manifest():
     reader = Reader.from_file("tests/fixtures/cloud.jpg")
     manifest = reader.get_active_manifest()
@@ -108,6 +116,8 @@ def sign(data: bytes) -> bytes:
         signer = create_signer(sign, SigningAlg.PS256, certs, "http://timestamp.digicert.com")
 
         builder = Builder(manifest_def)
+        
+        builder.add_ingredient_file(ingredient_def, data_dir + "A.jpg")
 
         builder.add_resource_file("A.jpg", data_dir + "A.jpg")
 
@@ -132,6 +142,8 @@ def sign(data: bytes) -> bytes:
         assert manifest,["format"] == "image/jpeg"
         # There should be no validation status errors
         assert manifest.get("validation_status") == None
+        assert manifest["ingredients"][0]["relationship"] == "parentOf"
+        assert manifest["ingredients"][0]["title"] == "A.jpg"
     except Exception as e:
         print("Failed to sign manifest store: " + str(e))
         exit(1)