Comments of keys not shown

[Hackerpcs]

As described here
#341 (comment)

until this build
https://github.com/dlech/KeeAgent/actions/runs/2290793867
I can see the comments of the keys

on the latest v0.13.1 I can't

[dlech]

Which private key file format are you using?

[Hackerpcs]

-----BEGIN OPENSSH PRIVATE KEY-----
foobarbaz
-----END OPENSSH PRIVATE KEY-----

[dlech]

KeeAgent currently uses only the public key comment. For this file format, the comment is encrypted, so it is only available after the private key has been decrypted. As a workaround, you might be able to add a .pub file containing the public key and the comment as an attachment so that KeeAgent will pick up the comment.

We could also consider adding a Comment property to SshPrivateKey (to be populated here) for private key comments and use this as a fallback when a public key comment is not provided.

[Hackerpcs]

My keys aren't encrypted, they are generated with this command without entering a passphrase

ssh-keygen -t rsa -b 4096 -C "comment" -f "key_rsa4096"

It worked till that version (which I currently use) though, couldn't it be done in the stable also like it was there?

[dlech]

Most people seem to use encrypted keys and decrypting the key even when it is not being used caused performance problems for newer keys formats with CPU intensive key derivation functions. Fixing that problem has lead to this problem.

We could add a hack that to the effect of "if the key is not encrypted, use the comment", but that would only work for people with unencrypted keys. Attaching a separate public key file that includes the comment is a more universal workaround.

[egfx-notifications]

I just updated from KeeAgent 0.12.1 to 0.13.1 and can confirm @Hackerpcs findings

I have both private and public keys in KeePass and configured for KeeAgent usage. Starting with 0.13.1 I no longer see the comment

• in the key selection screen I use when an identities list is prompted (Comment column is empty)
• in the confirmation prompt for key usage (it only displays key '', this is especially frustrating, as now I only have the fingerprint to check which key is about to be used)
• in the key usage windows notification (it only displays key '')

If I go back to KeeAgent 0.12.1 it works again.

[Hackerpcs]

Personally I use this build
https://github.com/dlech/KeeAgent/actions/runs/2290793867
that both works (regarding issue #341, Windows 10 21h2) and has comments.

[egfx-notifications]

Didn't manage to do a git bisect yet, but looking at the commits since 0.12.1 I consider this the most likely subject
1227354

[egfx-notifications]

Personally I use this build https://github.com/dlech/KeeAgent/actions/runs/2290793867 that both works (regarding issue #341, Windows 10 21h2) and has comments.

That would be two commits before the commit I suspect, so that sounds legit

[egfx-notifications]

KeeAgent currently uses only the public key comment. For this file format, the comment is encrypted, so it is only available after the private key has been decrypted. As a workaround, you might be able to add a .pub file containing the public key and the comment as an attachment so that KeeAgent will pick up the comment.

We could also consider adding a Comment property to SshPrivateKey (to be populated here) for private key comments and use this as a fallback when a public key comment is not provided.

@dlech Looking at the diff between 0.12.1 and 0.13.1 I think I understand what you meant with the encrypted comment. Unfortunately the public key attachment I also have in place is not helping because the code only ever checks for a public key attachment if the private key doesn't have a public key included. So my readable public key attachment is ignored, because a public key is already present in the private key, but I can't access the comment because it is encrypted.

[mplattner]

I have the same problem: since the update to v0.13.1.0 my keys' comments are not displayed anymore, beside for one old and used rsa1024 key. All keys (including the old one) are encrypted.

[attisan]

I have the same problem: since the update to v0.13.1.0 my keys' comments are not displayed anymore, beside for one old and used rsa1024 key. All keys (including the old one) are encrypted.

same here. in my case public key comments for ssh-ed25519 aren't showing up anymore

[KuttKatrea]

What about a setting to use the entry title as the comment? Is that feasible @dlech?

Wouldn't require a public key nor decrypting the private key

[dlech]

That sounds like a good idea for a fallback.

[KuttKatrea]

That sounds like a good idea for a fallback.

I created a PR:
#355