feat: sign protected pdfs #498

xhyrom · 2024-08-19T17:15:19Z

Fixes #495
Resolves #395

PDF/A detekcia vypnutá pri zaheslovanom dokumente source

taktiež to riešenie s vytvorením našeho "protected" dokumentu, avšak neviem, či je tu úprimne iné riešenie keďže to heslo musíme mať uložené pri dokumente

output.mp4

celuchmarek · 2024-08-20T06:32:16Z

src/main/java/digital/slovensko/autogram/ui/gui/MainMenuController.java

-                    new SaveFileResponder(file, autogram, userSettings.shouldSignPDFAsPades()),
-                    userSettings.isPdfaCompliance(), userSettings.getSignatureLevel(), userSettings.isEn319132(), tspSource, userSettings.isPlainXmlEnabled());
-            autogram.sign(job);
+            var document = SigningJob.createDSSFileDocumentFromFile(file);


Nebolo by lepšie to volať v tejto metóde?

a prečo to nedáme do SigningJob.build? To sa volá aj pre buildFromRequest aj pre buildFromFile.

keď potrebuješ tie getParametersForFile, tak by som to normláne osobitne zavolal v buildFromFile a buildFromRequest - nevadí, že to bude 2x, stále to bude lepšie ako teraz volané z GUI.

A neviem úplne, čo sa stane, keď pošleš taký súbor cez API - nebude už server chcieť niečo čítať z toho súboru?

Viem, že som to včera nespravil tak lebo SigningJob#build vracia SigningJob a ja Autogram#handleProtectedPdfDocument musím obaliť do work threadu (inak by to celé zamrzlo), teda tak či tak by sme to museli ešte vyššie obaliť v tomto jednom prípade ked signujeme jeden súbor, keďže pri hromadnom to obalené už je a pri CLI to netreba. Avšak bolo by to asi lepšie, zmazal by sa duplikačný kód 👍

keď potrebuješ tie getParametersForFile, tak by som to normláne osobitne zavolal v buildFromFile a buildFromRequest - nevadí, že to bude 2x, stále to bude lepšie ako teraz volané z GUI.

A neviem úplne, čo sa stane, keď pošleš taký súbor cez API - nebude už server chcieť niečo čítať z toho súboru?

Ešte stále je to draft, API budem riešiť dnes 😅

Nemáš nejaký lepší nápad ako sa zbaviť tohto?
https://github.com/slovensko-digital/autogram/pull/498/files#diff-daf36d389667b41baf5336c039d7fc72f9132298c577ef4ec11e45dd5fe54071R114
potrebujeme proste to obaliť v threade, keďže inak celá app zamrzne

moc pekné riešenie nemám. Ale teda dá sa poslať GUI inštancia do MainMenuControllera a potom volať rovno gui.onWorkThreadDo a nemusíš tam mať tú obaľovaciu metódu cez Autogram.

ale teda tento handleProtectedPdfDocument bude musieť ostať takto ešte pred vzniknutím SigningJobu, lebo obsah pdf potrebujeme čítať ešte pri vyrábaní parametrov. Otázka je, či to vyhodíme z tohto buildu a spravíme to ešte niekde predtým.

@xhyrom tuto ešte asi posledná vec. Pre SigningJob máme 2 buildre buildFromRequest a buildFromFile. Do buildFromRequest už ide AutogramDocument s nastaveným heslom ak treba. Avšak do buildFromFile ešte nejde, AutogramDocument sa vyrába v ňom a musí sa v ňom ešte volať autogram.handleProtectedPdfDocument(document).

V prvom rade, ten buildFromRequest už nemá veľmi zmysel, lebo len zavolá private build. Tým pádom tu máme jediný špeciálny buidler tento druhý, ktorý do seba ťahá Autogram. Navrhujem build spraviť public, vyhodiť buildFromRequest a premiestniť buildFromFile do Autogramu. Tým pádom bude závislosť jendosmerná - Autogram bude používať SigningJob - teraz je to obojsmerné, lebo Autogram aj tak používa SigningJob, ale v tomto builderi musí aj SignignJob používať Autogram.

celuchmarek · 2024-08-20T06:45:52Z

@xhyrom ale mám pocit, že zabúdaš, že PDF môže byť "zaheslované" dvoma spôsobmi - encrypted vs password protected. Encrypted ani neprečítaš. Password protected sa zobrazí, ale nedá sa upravovať (napr. nejaký formulár v ňom) iba s heslom.

xhyrom · 2024-08-20T16:06:24Z

@xhyrom ale mám pocit, že zabúdaš, že PDF môže byť "zaheslované" dvoma spôsobmi - encrypted vs password protected. Encrypted ani neprečítaš. Password protected sa zobrazí, ale nedá sa upravovať (napr. nejaký formulár v ňom) iba s heslom.

Áno, máš pravdu. Zatiaľ neriešim správne problém ktorý sa vyskytol presne v #495

Vďaka pdftk sa mi ale podarilo reprodukovať problém - pdftk unprotected.pdf output perms-for-editing.pdf owner_pw heslo123

xhyrom · 2024-08-20T16:14:44Z

Tento jeden check by to mal riešiť:
bb7c5a1#diff-7e5765eb3674f6f996bff11d3266f78fdc3a4205e7a4bdf33545559517a8da62R19-R22

celuchmarek · 2024-08-22T07:47:44Z

Okay teda. A keď hodíš encrypted alebo password protected PDF do Autogramu, tak ťa to v oboch prípadoch vyzve na zadanie hesla a potom to funguje ďalej? Hoď aj nejaké obrázky alebo video, pls.

xhyrom · 2024-08-22T07:49:05Z

Okay teda. A keď hodíš encrypted alebo password protected PDF do Autogramu, tak ťa to v oboch prípadoch vyzve na zadanie hesla a potom to funguje ďalej? Hoď aj nejaké obrázky alebo video, pls.

Áno, presne tak. Hodím keď budem na kompe 👍

xhyrom · 2024-08-22T19:30:55Z

output.mp4

celuchmarek · 2024-08-26T08:08:53Z

Ok, super. Ešte si to vyskúšam lokálne. Vieš sem niekde hodiť tie sample PDF súbory s heslami, aby som už nemusel vyrábať ďalšie?

Navyše, pri PDF uzamknutých proti zmenám, je potrebné zadávať heslo aj pre zobrazenie dokumentu? Rozmýšľam, či v tom prípade nepýtať heslo až pri podpisovaní.

xhyrom · 2024-08-26T09:52:39Z

Ok, super. Ešte si to vyskúšam lokálne. Vieš sem niekde hodiť tie sample PDF súbory s heslami, aby som už nemusel vyrábať ďalšie?

Navyše, pri PDF uzamknutých proti zmenám, je potrebné zadávať heslo aj pre zobrazenie dokumentu? Rozmýšľam, či v tom prípade nepýtať heslo až pri podpisovaní.

Hneď ako budem môcť tak ich sem poposielam, čo sa týka toho zobrazenia, môžeme to spraviť, ale v podstate je to podľa mňa zbytočné.

xhyrom · 2024-08-27T13:15:43Z

unprotected.pdf
perms-for-editing.pdf (heslo123)
test-protected.pdf (heslo123, toto je enkryptované)

src/main/java/digital/slovensko/autogram/model/ProtectedDSSDocument.java

src/main/java/digital/slovensko/autogram/util/PDFUtils.java

src/main/java/digital/slovensko/autogram/core/SigningJob.java

src/main/java/digital/slovensko/autogram/core/SignatureValidator.java

src/main/java/digital/slovensko/autogram/core/eforms/xdc/XDCBuilder.java

src/main/java/digital/slovensko/autogram/server/SignEndpoint.java

src/main/java/digital/slovensko/autogram/ui/BatchGuiFileResponder.java

src/main/java/digital/slovensko/autogram/ui/gui/GUI.java

src/main/java/digital/slovensko/autogram/util/AsicContainerUtils.java

celuchmarek

nechal som nejaké commenty. Hlavne treba dorieišť, čo s tými dvoma heslami a ProtectedDocument zjednodušiť do jednej classy.

src/main/resources/digital/slovensko/autogram/ui/gui/password-dialog.fxml

src/main/java/digital/slovensko/autogram/model/AutogramDocument.java

src/main/java/digital/slovensko/autogram/core/SigningJob.java

celuchmarek · 2024-10-09T12:51:20Z

@xhyrom už to vyzerá dosť dobre. Pozri tých pár komentov. A potom ešte z testovania som narazil na error s batchom. Ak som zadal zlé heslo na niektorom dokumente, vrátilo to nepekný error a celý batch sa zastavil a nejako divne zasekol. Vieš to pozrieť? Neviem, či to je iba týmito heslami alebo tam je iná chyba.

digital.slovensko.autogram.core.errors.AutogramException: eu.europa.esig.dss.pades.exception.InvalidPasswordException: Encrypted document : Cannot decrypt PDF, the password is incorrect
	at digital.slovensko.autogram.core.AutogramBatchStartCallback.handleException(AutogramBatchStartCallback.java:35)
	at digital.slovensko.autogram.core.AutogramBatchStartCallback.accept(AutogramBatchStartCallback.java:26)
	at digital.slovensko.autogram.core.AutogramBatchStartCallback.accept(AutogramBatchStartCallback.java:9)
	at digital.slovensko.autogram.ui.gui.BatchDialogController.lambda$onMainButtonPressed$3(BatchDialogController.java:92)
	at java.base/java.lang.Thread.run(Thread.java:833)
Caused by: eu.europa.esig.dss.pades.exception.InvalidPasswordException: Encrypted document : Cannot decrypt PDF, the password is incorrect
	at eu.europa.esig.dss.pdf.pdfbox.PdfBoxDocumentReader.<init>(PdfBoxDocumentReader.java:115)
	...
	at eu.europa.esig.dss.validation.SignedDocumentValidator.validateDocument(SignedDocumentValidator.java:356)
	at digital.slovensko.autogram.core.SignatureValidator.getSignedDocumentSimpleReport(SignatureValidator.java:175)
	at digital.slovensko.autogram.core.SigningJob.getParametersForFile(SigningJob.java:200)
	at digital.slovensko.autogram.core.SigningJob.buildFromFile(SigningJob.java:195)
	at digital.slovensko.autogram.ui.BatchGuiFileResponder.onBatchStartSuccess(BatchGuiFileResponder.java:67)
	at digital.slovensko.autogram.core.AutogramBatchStartCallback.handleSuccess(AutogramBatchStartCallback.java:42)
	at digital.slovensko.autogram.core.AutogramBatchStartCallback.accept(AutogramBatchStartCallback.java:24)
	... 3 more

xhyrom · 2024-10-09T18:23:38Z

@xhyrom už to vyzerá dosť dobre. Pozri tých pár komentov. A potom ešte z testovania som narazil na error s batchom. Ak som zadal zlé heslo na niektorom dokumente, vrátilo to nepekný error a celý batch sa zastavil a nejako divne zasekol. Vieš to pozrieť? Neviem, či to je iba týmito heslami alebo tam je iná chyba.
``` digital.slovensko.autogram.core.errors.AutogramException: eu.europa.esig.dss.pades.exception.InvalidPasswordException: Encrypted document : Cannot decrypt PDF, the password is incorrect at digital.slovensko.autogram.core.AutogramBatchStartCallback.handleException(AutogramBatchStartCallback.java:35) at digital.slovensko.autogram.core.AutogramBatchStartCallback.accept(AutogramBatchStartCallback.java:26) at digital.slovensko.autogram.core.AutogramBatchStartCallback.accept(AutogramBatchStartCallback.java:9) at digital.slovensko.autogram.ui.gui.BatchDialogController.lambda$onMainButtonPressed$3(BatchDialogController.java:92) at java.base/java.lang.Thread.run(Thread.java:833) Caused by: eu.europa.esig.dss.pades.exception.InvalidPasswordException: Encrypted document : Cannot decrypt PDF, the password is incorrect at eu.europa.esig.dss.pdf.pdfbox.PdfBoxDocumentReader.(PdfBoxDocumentReader.java:115) ... at eu.europa.esig.dss.validation.SignedDocumentValidator.validateDocument(SignedDocumentValidator.java:356) at digital.slovensko.autogram.core.SignatureValidator.getSignedDocumentSimpleReport(SignatureValidator.java:175) at digital.slovensko.autogram.core.SigningJob.getParametersForFile(SigningJob.java:200) at digital.slovensko.autogram.core.SigningJob.buildFromFile(SigningJob.java:195) at digital.slovensko.autogram.ui.BatchGuiFileResponder.onBatchStartSuccess(BatchGuiFileResponder.java:67) at digital.slovensko.autogram.core.AutogramBatchStartCallback.handleSuccess(AutogramBatchStartCallback.java:42) at digital.slovensko.autogram.core.AutogramBatchStartCallback.accept(AutogramBatchStartCallback.java:24) ... 3 more ```

malo by byť opravené

xhyrom added 4 commits August 19, 2024 19:13

feat: initial implementation of signing protected pdfs

ff84ad3

feat: allow protected pdfs in batch

6d00beb

fix: proxy asic original document instead casting

84a311f

fix: tests

6eed34d

celuchmarek reviewed Aug 20, 2024

View reviewed changes

fix: run callback, default pass

aad94b7

fix: check if creating signature field is possible

bb7c5a1

refactor: change place where we're asking for password

acb36d4

fix: tests

11ef8b9

xhyrom marked this pull request as ready for review August 27, 2024 13:15