I am using v3.0.2.
Let's say you have a file named A.exe and in the same folder a signature file named A.exe.asc.
If you right click on A.exe and do More GpgEX options -> Verify you get an error dialog reading:
Error returned by the GnuPG user interface (Kleopatra):
Internal error
If you do the same on A.exe.asc, verification works fine.
v2.3.3 verifies correctly the file even if you right click on A.exe instead of the signature file.
This is a regression.
| Status | Assigned | Task | ||
|---|---|---|---|---|
| Resolved | • aheinecke | T3607 Gpg4win 3.0.3 | ||
| Resolved | • aheinecke | T3658 GpgEX: internal error when trying to verify a non-signature file |
Thank you for your report. I can reproduce this problem. Kleopatra correctly looks for the signature file but then fails to set the protocol. This results in an internal error.
Will be fixed.
Fixed with https://commits.kde.org/kleopatra/066c6e4ca82d064437f1902838d09fa903147e40
will be part of the next release.
@aheinecke thanks for the fix. But I have a suggestion for the code(I only looked at the diff):
Why not use the variable you created earlier?
So instead of
t->setProtocol(findProtocol(classification));
do
t->setProtocol(proto);
It seems simpler and more efficient, than performing the lookup again.
Indeed, thanks for the note. I added the variable only later on for the check of protocol unknown and overlooked to update the setProtocol call.
I've updated the code accordingly.