FOP-3299: Remove DigestMethod from pdf signing

simonsteiner1984 · simonsteiner1984 · commit e355d4ea8716 · 2026-03-24T13:15:46.000Z
diff --git a/fop-core/src/main/java/org/apache/fop/pdf/PDFSignature.java b/fop-core/src/main/java/org/apache/fop/pdf/PDFSignature.java
@@ -76,7 +76,6 @@ static class SigRef extends PDFDictionary {
         SigRef() {
             put("Type", new PDFName("SigRef"));
             put("TransformMethod", new PDFName("DocMDP"));
-            put("DigestMethod", new PDFName("SHA1"));
             put("TransformParams", new TransformParams());
         }
     }
diff --git a/fop-core/src/test/java/org/apache/fop/pdf/PDFSigningTestCase.java b/fop-core/src/test/java/org/apache/fop/pdf/PDFSigningTestCase.java
@@ -52,6 +52,7 @@ public void testFO() throws Exception {
                 + "  /T (Signature1)\n"
                 + "  /TU (Signature1)\n"
                 + "  /Rect [0 0 0 0]"));
+        Assert.assertFalse(out.toString().contains("/DigestMethod"));
     }
 
     @Test

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,6 @@ static class SigRef extends PDFDictionary {`
`76`	`76`	`SigRef() {`
`77`	`77`	`put("Type", new PDFName("SigRef"));`
`78`	`78`	`put("TransformMethod", new PDFName("DocMDP"));`
`79`		`- put("DigestMethod", new PDFName("SHA1"));`
`80`	`79`	`put("TransformParams", new TransformParams());`
`81`	`80`	`}`
`82`	`81`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,6 +52,7 @@ public void testFO() throws Exception {`
`52`	`52`	`+ " /T (Signature1)\n"`
`53`	`53`	`+ " /TU (Signature1)\n"`
`54`	`54`	`+ " /Rect [0 0 0 0]"));`
	`55`	`+ Assert.assertFalse(out.toString().contains("/DigestMethod"));`
`55`	`56`	`}`
`56`	`57`
`57`	`58`	`@Test`