-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AG-6 Add invalid certificate warning to certificate picker #297
AG-6 Add invalid certificate warning to certificate picker #297
Conversation
@jsuchal spravil som to tak, že Autogram samotný povoľuje podpis ekspirovaným certifikátom, ale GUI ti ho dá na výber, iba ak to máš zapnuté a upozorní ťa. CliUI ti vždy pri nich napíše, že sú ekspirované (možno tiež pridať flag do CLI, bez ktorého ich bude ignorovať). Skúšal som ten parameter nejako dostať do SigningJobu (tam sa to povoľuje), ale to je také zlé previazanie, že UserSettings platné iba pre GUI by šli až niekam do Autogramu (ten volá sign). Dáva mi zmysel, aby sme zjednotili UserSettings a CliParamters cez nejaký interface Settings a ponúkali by rovnaké atribúty. S tým, že UserSettings platia pre GUI a CliParameters platia per spustenie pre CLI. Tým pádom som ten test dal preč. Problém je, že otestavať by teraz bolo treba GUI a pozerať, aká okno otvorí a podobne. A to je vec, čo nevieme spraviť. |
|
||
public class NoValidKeysDetectedException extends AutogramException { | ||
public NoValidKeysDetectedException() { | ||
super("Nastala chyba", "Nenašli sa žiadne platné podpisové klúče", "Na karte sa pravdepodobne nenachádzajú žiadne platné klúče, ktoré by sa dali použiť na podpisovanie. Boli však nájdené ekspirované kľúče, ktorými je možné podpisovať až po zmene v nastaveniach.\n\nV prípade nového občianskeho preukazu to môže znamenať, že si potrebujete certifikáty na podpisovanie cez občiansky preukaz vydať. Robí sa to pomocou eID klienta.", null); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
super("Nastala chyba", "Nenašli sa žiadne platné podpisové klúče", "Na karte sa pravdepodobne nenachádzajú žiadne platné klúče, ktoré by sa dali použiť na podpisovanie. Boli však nájdené ekspirované kľúče, ktorými je možné podpisovať až po zmene v nastaveniach.\n\nV prípade nového občianskeho preukazu to môže znamenať, že si potrebujete certifikáty na podpisovanie cez občiansky preukaz vydať. Robí sa to pomocou eID klienta.", null); | |
super("Nastala chyba", "Nenašli sa žiadne platné podpisové certifikáty", "V úložisku certifikátov sa pravdepodobne nenachádzajú žiadne platné podpisové certifikáty, ktoré by sa dali použiť na podpisovanie. Boli však nájdené ekspirované certifikáty, ktorými je možné podpisovať až po zmene v nastaveniach.\n\nV prípade nového občianskeho preukazu to môže znamenať, že si potrebujete certifikáty na podpisovanie cez občiansky preukaz vydať. Robí sa to pomocou obslužného softvéru eID klient.", null); |
Ešte to nevaliduje správne, ale chcem vyriešiť UX, tak vytváram PR.
Ktorý z potenciálnych návrhov, prípadne ako inak?