search

slovensko-digital / autogram

Nový, krajší a lepší nástroj na podpisovanie.
European Union Public License 1.2
109 stars 20 forks source link

Prejst na checked exceptions #186

Open jsuchal opened 1 year ago

jsuchal commented 1 year ago

Z diskusie na slacku vyplynulo, ze best practice su checked exceptions. Co som robil maly pokus, tak by to nemuselo byt take tazke dodrzat, lebo my exceptions dokonca vraciame (ako error).

@sabomichal vedel by si sa na toto pozriet?

sabomichal commented 1 year ago

skusim

sabomichal commented 1 year ago

Povodny PR som po refaktore v kode zavrel, nakolko viacero veci sa zmenilo, nasledne som sa k tomu uz z nedostatku casu nevratil. Kazdopadne nakoniec asi fajn, pretoze to aj tak nebolo uplnym riesenim - to vyzaduje vyriesenie narabania s eu.europa.esig.dss.model.DSSDocument resp. celym DSS API, ktore pri pouziti moze kdekolvek vyhodit unchecked eu.europa.esig.dss.model.DSSException. Tie potom treba odchytavat na viacerych miestach.

Aby bolo preto mozne zaviest checked exceptions, navrhujem preto najskor spravit refaktor, ktory pouzitie DSS API a API Autogramu oddeli. Vo vysledku tak idealne budem vynimky z DSS API vediet odchytit na 1 mieste.