Skip to content
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

Conversation

celuchmarek
Copy link
Member

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?

Screenshot from 2023-10-11 16-24-03

@celuchmarek
Copy link
Member Author

@jsuchal @tajovic @TomasMakan

@jsuchal
Copy link
Member

jsuchal commented Oct 12, 2023

Skusil som to v govuk design manuale podla toho co oni pouzivaju a pride mi, ze toto je take nenasilne.
image

@celuchmarek
Copy link
Member Author

Screenshot from 2023-10-12 12-39-10
Screenshot from 2023-10-12 12-38-14

@celuchmarek
Copy link
Member Author

@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ť.

@celuchmarek celuchmarek requested a review from jsuchal October 12, 2023 11:03

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);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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);

@celuchmarek celuchmarek merged commit 70b9b6b into slovensko-digital:main Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants