Signature validation #214

celuchmarek · 2023-06-28T06:34:09Z

No description provided.

jsuchal · 2023-07-19T07:54:40Z

@celuchmarek ku GUI mame nejake vlakno na slacku, pripadne Taja to videl?

celuchmarek · 2023-07-19T07:56:30Z

@jsuchal jo, Taja to videl, ale nemal nič veľmi k tomu. Na slacku vo vlákne sú aktuálnejšie screenshoty.

celuchmarek · 2023-07-19T14:42:13Z

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

+            journalCertificateSource = new KeyStoreCertificateSource(
+                new File("/home/turtle/slovensko_digital/autogram/src/main/resources/lotlKeyStore.p12"), "PKCS12", "dss-password");


Toto je pomerne divná vec. List of Trusted Lists (napr pre EU) obyčajne nie je nijako trusted, keď to iba stiahneme z nejakej adresy. Preto existuje na internete 8 certifikátov tohto LOTL. Aj podľa DSS oficiálnej dokumentácie s examplami sa to má robiť tak, že do tohto keystoru človek uloží tých 8 certifikátov, ktorými sa validuje samotný LOTL. (google Pivot LOTL)

Tak som manuálne z oficiálnej stránky okopíroval tie certifikáty v textovej forme a nakoniec ich importol do tohoto keystoru. A takto to funguje. LOTL je teraz "well signed". Avšak, zdá sa mi divné, že keystore je chránený nejakým heslom, ktoré je public. Ale možno je tam to heslo iba preto, lebo to Java nevie bez hesla.

…ature-validation

celuchmarek · 2023-08-10T07:01:15Z

@jsuchal Tuto je zatiaľ validácia bez GUI. Je zatiaľ otázka, ako to bude fungovať ohľadom ukladania TL a čítania certifikátov na iných strojoch, ale uvidíme. Tie controllery som vytváral trochu inak, aby som vedel rozumne držať ich inštancie a posielať do nich potom niekedy v čase tie validačné reporty. To ale nefunguje úplne dobre, lebo, ak je otvorený zoznam podpisov, nedá sa klikať na okno podpisovania a podobne. Niekedy (10%) sa okná otvoria v zlom poradí, že okno podpisovania je vždy vpredu a ostatné sú za tým a nedajú sa dostať pred okno podpisovania.

jsuchal · 2023-08-11T14:56:44Z

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

+        -> ui.startSigning(job, this));
+
+        ui.onWorkThreadDo(()
+        -> checkAndValidateSignatures(job));


Toto sa ma diat aj pri CLI? Nemyslim uplne.

Dal som do startSigning do vnútra GUI.

jsuchal · 2023-08-11T14:57:29Z

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

+        ui.onWorkThreadDo(() -> {
+            SignatureValidator.getInstance().initialize();
+        });
+
+        var timer = new Timer();
+        timer.scheduleAtFixedRate(new TimerTask() {
+                public void run() {
+                    SignatureValidator.getInstance().refresh();
+                }
+            },
+             3600000L,
+             3600000L);
+
+        return timer;


Toto robi co?

Každých 8 hodín behu Autogramu refreshne Trusted Listy

jsuchal · 2023-08-11T14:59:30Z

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

+    private Reports signatureCheckReport;
+    private Reports signatureValidationReport;


Toto mi tu domenovo uplne nesedi, Job bol immutable a report s nim asi nesuvisi ci?

No report je výsledok jobu a job chceme spúšťať iba raz, lebo trvá nejaký čas. Takže si ho uložíme sem.

Podobne sme to mali s vizualizaciou a netlacime to do jobu

Okay. Zapracujem budúci týždeň

Toto som prehodil do "pamäte" Controllera.

jsuchal · 2023-08-11T15:00:01Z

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

+    public void onSignatureValidationCompleted(SigningJob job);
+
+    public void onSignatureCheckCompleted(SigningJob job);
+


Tu radsej poslime ten report ako separe parameter ako to posuvat cez signing job

@jsuchal Podľa signingJobu sa potom nájde príslušný controller, takže asi to nepôjde bez toho.

Mozeme to spravit naopak. Report bude obsahovat odkaz na signingjob a uz vies.

Report je ale objekt z DSS. Chceme nad tým vytvoriť ďalší wrapper?

Rozumiem a hej, dava mi to zmysel.

Vyrobil som core.ValidationReportsWrapper

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

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

jsuchal · 2023-08-11T15:05:51Z

src/main/resources/digital/slovensko/autogram/ui/gui/idsk.css

+  -fx-background-color: #cce2d8;
+  -fx-border-color: #cce2d8;
+}
+
+.autogram-tag-invalid {
+  -fx-background-color: #f8d7da;
+  -fx-border-color: #f8d7da;
+}
+
+.autogram-tag-valid--text {
+  -fx-text-fill: #005a30;
+  -fx-fill: #005a30;
+}
+
+.autogram-tag-invalid--text {
+  -fx-text-fill: #721c24;
+  -fx-fill: #721c24;


Farby potom vytiahnime do konstant

Ďalej pôjdem robiť GUI, tak pri tej príležitosti to povyťahujem.

jsuchal · 2023-08-17T15:08:31Z

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

+        if (!SignatureValidator.hasSignatures(job.getDocument()))
+            return;
+
+        var reports = SignatureValidator.getInstance().getSignatureValidationReport(job.getDocument());


Nedava zmysel, aby nas validator dostal job a vratil uz ten ValidationReports? (Wrapper by som dropol, ved to je nas namespace)

jsuchal · 2023-08-17T15:11:57Z

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

@@ -31,6 +32,8 @@ public void start(Stage windowStage) throws Exception {

        windowStage.setOnCloseRequest(event -> {
            new Thread(server::stop).start();
+            timer.cancel();


Toto mi pride ako prilisna vnutornost. Nemal by mat mat autogram len nejaky close kde sa toto udeje?

tomuto nerozumiem. Vytvára a zastavuje sa to na rovnakom mieste ako server. To nie je ok?

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

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

src/main/resources/digital/slovensko/autogram/server/server.yml

celuchmarek · 2023-09-21T16:30:26Z

@tajovic @jsuchal Už je to v takom stave, že by som sa nehanbil to mergnúť. Nakoniec som ten redizajn dokončil takto, že link som dal osobitne pod tabuľku.

Warning je potom takto:

celuchmarek · 2023-09-21T16:33:12Z

@jsuchal snažil som sa prerobiť aj flow podpisovania, ako sme sa bavili, ale radšej by som to spravil v osobitnom PR. Rovnako pred releasom by sme podľa mňa mali dorobiť aj nastavenia. Dám k tomu dokopy ešte nejaký návrh a postúpim ho na ux oddelenie.

jsuchal · 2023-09-21T16:36:36Z

@celuchmarek to UI este ja nieco skusim, mam napad.

co sa tyka prerabky flowu, suhlasim, ze to dajme bokom. settings nutne do release netreba, ale nekvasme dlho s tym PR lebo sa nam to rozlezie.

celuchmarek · 2023-09-21T16:40:37Z

Nerozumiem poslednej vete.

jsuchal · 2023-09-21T16:41:08Z

Ze ten settings PR riesme dalej, ale nutne do release nam ho myslim netreba.

jsuchal · 2023-09-22T15:42:16Z

@celuchmarek taketo sa mi podarilo

Matie ma to Vysledok overenia: neznamy podpis.

celuchmarek · 2023-09-22T15:55:55Z

To je ten revokovaný OP?

jsuchal · 2023-09-22T16:14:09Z

Neviem ci je revokovany ale toto je detail

celuchmarek · 2023-09-22T16:18:11Z

Aha, čiže v sľubovanom čase podpisu bol ešte certifikát validný. Teraz je už ekspirovaný a nemá pečiatku, ktorá by dosvedčila ten čas. Tým pádom sa nedá zaručiť, že je to platné a že podpis hovorí pravdu.

Ale hej, Neznámy popis by v tomto prípade bolo dobré zmeniť na niečo iné.

…table

celuchmarek · 2023-09-28T13:27:18Z

@jsuchal tak teda už som nejako dokončil ten zoznam podpisov. Je tam tiaž taký divný hack, že treba zavolať stage.sizeToScene, setManaged(true) a zase stage.sizeToScene. Inak sa neresizol správne mainBox a pôvodí to z toho GridPanu. Možno ešte otázka, či nedať max 3 a nie max 4 podpisy.

Pridal som aj wrapovanie do vn[tra badgu/tagu (máme najviac 45 znakov) a tiež som trochu pofixoval aj wrapovanie viacerých badgov. To funguje pekne v signing dialogu v grid pane, ale nefunguje natívne v detailoch podpisov - tam je napevno 300px, ale tam sa okno nedá resizovať, takže asi ok.

Trochu viac som ohandloval aj tie hlášky "výsledok overenia".

jsuchal · 2023-09-29T12:58:34Z

@celuchmarek uz sme skoro tam, este updatnime release checklist kedze pridavame riadne velku funkcionalitu.

celuchmarek · 2023-09-29T13:21:18Z

Ešte tam bolo niečo čarovné. Zle sa zobrazili tie nižšie badge, ak ten horný pretekal. Ak som ho zabalil, tak sa to zase kazilo ak pretekalo meno. Keď som zavolal dvakrát stage.show(), už to ide ok vo všetkých prípadoch :D Do checklistu chceme dať aj nejaké sample podpísané súbory na testovanie overenia a podobne alebo len, že to vie overiť nejaké podpisy?

… AG-85/signature-validation

jsuchal · 2023-09-29T13:27:47Z

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

@@ -133,7 +133,7 @@ public static VBox createSignatureBox(Reports reports, boolean isValidated, Stri
        if (!isValidated)
            badge = SignatureBadgeFactory.createInProgressBadge();
        else if (isFailed)
-            badge = SignatureBadgeFactory.createInvalidBadge("Neplatný podpis");
+            badge = SignatureBadgeFactory.createInvalidBadge("Neplatný podpis Neplatný podpis Neplatný podpis Neplatný podpis");


jsuchal · 2023-09-29T13:28:33Z

Nejaky sample podpisany by sa hodil ale neviem kam ho dame. Niekam ho treba asi zavesit do nejake issue nech sa nestrati.

celuchmarek added 6 commits June 28, 2023 08:31

wip validation

b10b052

wip validation

3989444

add wip signature check and validation visualization

8d1b71b

gui major update

3ebcfdc

rm unintentedly commited files

e92c66b

fix ocsp header and add lotl certs

1c85504

minor refactor

0ec71a1

celuchmarek commented Jul 19, 2023

View reviewed changes

celuchmarek added 4 commits July 26, 2023 08:56

refactor and speed up

7f8719a

Merge remote-tracking branch 'slovensko-digital/main' into AG-85/sign…

f2fb6fd

…ature-validation

make SignatureValidation Singleton

ced89e3

detect signature qualification and render the results

279fd7c

celuchmarek force-pushed the AG-85/signature-validation branch from b282a5c to 279fd7c Compare August 1, 2023 13:36

celuchmarek added 6 commits August 1, 2023 16:12

merge main

8e4ac87

add ts qualification check

0a250ba

validate api wip

61e98f1

revert addition of validaiton api

3124176

refactor

b263d31

rm unwanted file

f30af12

celuchmarek marked this pull request as ready for review August 10, 2023 06:56

celuchmarek requested a review from jsuchal August 10, 2023 06:56

jsuchal requested changes Aug 11, 2023

View reviewed changes

fix resource packaging and small fixes

841863e

celuchmarek requested a review from a team as a code owner August 17, 2023 12:31

refactor

313e6c7

jsuchal requested changes Aug 17, 2023

View reviewed changes

rm server.yml from PR

291bae8

celuchmarek added 2 commits September 21, 2023 16:00

minor refactor and cleanup

514e1b5

mv link from whoSigned to a new link button

3310fd0

only show validation section when signatures are present

19e37cc

celuchmarek added 4 commits September 27, 2023 09:35

wip TableView

4b00f88

max N signatures in table

340f8fb

refactor hidden signatures and make tags wrap text

2ffd3bb

do not set multiple badge wrapping width in signing dialog signature …

6420582

…table

jsuchal and others added 3 commits September 29, 2023 11:09

Minor refactorings, text tuning a bugfixes

8b28106

Minor wording fix

ea1b206

fix present signatures screen width

a1b7f6a

fix signatures controller rendering

e2855d2

jsuchal added 2 commits September 29, 2023 15:26

Readme tuning

e743554

Merge remote-tracking branch 'origin/AG-85/signature-validation' into…

74a857e

… AG-85/signature-validation

jsuchal reviewed Sep 29, 2023

View reviewed changes

celuchmarek added 2 commits September 29, 2023 15:42

rm debug label

4dcf4e6

add signature validation to release checklist

2fc3a0d

celuchmarek merged commit 8f57bb3 into slovensko-digital:main Sep 29, 2023

		journalCertificateSource = new KeyStoreCertificateSource(
		new File("/home/turtle/slovensko_digital/autogram/src/main/resources/lotlKeyStore.p12"), "PKCS12", "dss-password");

		private Reports signatureCheckReport;
		private Reports signatureValidationReport;

		public void onSignatureValidationCompleted(SigningJob job);

		public void onSignatureCheckCompleted(SigningJob job);

Signature validation #214

Signature validation #214

Conversation

celuchmarek commented Jun 28, 2023 • edited Loading

jsuchal commented Jul 19, 2023

celuchmarek commented Jul 19, 2023

celuchmarek Jul 19, 2023 • edited Loading

Choose a reason for hiding this comment

celuchmarek commented Aug 10, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

celuchmarek commented Sep 21, 2023

celuchmarek commented Sep 21, 2023

jsuchal commented Sep 21, 2023

celuchmarek commented Sep 21, 2023

jsuchal commented Sep 21, 2023

jsuchal commented Sep 22, 2023

celuchmarek commented Sep 22, 2023

jsuchal commented Sep 22, 2023

celuchmarek commented Sep 22, 2023

celuchmarek commented Sep 28, 2023

jsuchal commented Sep 29, 2023

celuchmarek commented Sep 29, 2023

Choose a reason for hiding this comment

jsuchal commented Sep 29, 2023

celuchmarek commented Jun 28, 2023 •

edited

Loading

celuchmarek Jul 19, 2023 •

edited

Loading