kaikramer / keystore-explorer

KeyStore Explorer is a free GUI replacement for the Java command-line utilities keytool and jarsigner.
https://keystore-explorer.org/
GNU General Public License v3.0
1.67k stars 271 forks source link

[Wanted feature] Request for `List Certificates` enhancement #377

Closed The-Lum closed 1 year ago

The-Lum commented 2 years ago

This is a feature request.

Inspired by a screen of https://github.com/kaikramer/keystore-explorer/issues/375#issue-1281335966

`List Certificates`

Is your feature request related to a problem? Please describe. I'm always frustrated when I list certificates... I need sometime other fields, and want to filter or sort some fields...

Describe the solution you'd like For a future enhancement [on DListCertificatesKS.java], for the List Certificates screen perhaps adding:

Then also: adding possibility to filter or sorting all the different fields...

See also similar pseudo-request here:

We could even filter and show only those that have been signed by the sub-AC, with which we are about to sign the CRL...

If that interest other people... or if someone can propose a PR.. about List Certificates screen... Regards.

[FYI @kaikramer, @jgrateron]

jgrateron commented 2 years ago

Hi, indeed more functionality is needed on that screen. I am going to take some time to make those adjustments.

Regards.

jgrateron commented 2 years ago

Hello It uses the same component to display the entries of a keystore, this component also allows sorting and its columns depend on general preferences.

image

image

https://github.com/jgrateron/keystore-explorer/tree/enhanceListCertificate

Filtering I don't think I will perform it.

The-Lum commented 2 years ago

Hello @jgrateron, and all,

Here are simple (and easy [Sorry!]) remarks:

  1. Goods enhancement, futhermore that uniformise kse windows. :+1:
  2. How to avoid to duplicate code? Is it possible to factorise the code of the main windows and the code of the sub windows... (unfortunately I am not a good java dev)...

Regards.

jgrateron commented 2 years ago

Hello @The-Lum If you mean refactoring the methods setColumnsToIconSize and colAdjust, yes it is possible. It only remains that @kaikramer gives his approval of the improvement.

Regards

The-Lum commented 2 years ago

If you mean refactoring the methods setColumnsToIconSize and colAdjust, yes it is possible. It only remains that @kaikramer gives his approval of the improvement.

Yes. @kaikramer: Are you agree, if will setColumnsToIconSize and colAdjust be factorised? Is it necessary to create a new public class for that or link/include to KseFrame.java and change the 2 method in order to be public...


https://github.com/kaikramer/keystore-explorer/blob/78cf949a26d1a5d17fbc04d15df4f7ed01ba2373/kse/src/org/kse/gui/KseFrame.java#L2445 And: https://github.com/jgrateron/keystore-explorer/blob/75d11485320a055fbba9102fe1bdc508dfc476a2/kse/src/org/kse/gui/dialogs/sign/JListCertificates.java#L142


https://github.com/kaikramer/keystore-explorer/blob/78cf949a26d1a5d17fbc04d15df4f7ed01ba2373/kse/src/org/kse/gui/KseFrame.java#L3141 And:

https://github.com/jgrateron/keystore-explorer/blob/75d11485320a055fbba9102fe1bdc508dfc476a2/kse/src/org/kse/gui/dialogs/sign/JListCertificates.java#L152


Regards.

kaikramer commented 2 years ago

refactoring the methods setColumnsToIconSize and colAdjust, yes it is possible. It only remains that @kaikramer gives his approval of the improvement.

Yes, of course, it is a very good idea to reuse the component.

jgrateron commented 2 years ago

Hi Refactoring of methods setColumnsToIconSize and colAdjust to class JKseTable completed.

Regards.

image

The-Lum commented 1 year ago

Hi @kaikramer, @jgrateron, and all,

Thanks for this enhancement (on v5.5.2). That is consistent in order to have the same screen as the main screen. But:

See screenshot here (with red arrows): 2023-01-23 20_11_57-ED uber - KeyStore Explorer 5 5 2 arrows

In fact, the main question will be:


Thanks a lot. Regards, Th.

kaikramer commented 1 year ago

Hm, I see, it seems necessary to be able to enlarge the dialog now that we can have so many columns in this dialog...

What would be the minimum acceptable size (of the List Certificates windows)? depending or not on the contents of the table according to the preferences of the user...

Yes, good point, not sure if it is necessary to calculate the starting (or minimum) size... It seems like it should be quite a bit wider in your screenshot...

Is it necessary to open a new issue or not?

Yes, please, it helps me to keep an overview. It is fine if you keep it very short (just the screenshot and a headline).