By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 2883

Summary: tdepim/libkpgp the GPG part needs update to gpg2
Product: TDE Reporter: deloptes
Component: tdepimAssignee: Timothy Pearson <kb9vqf>
Status: RESOLVED FIXED    
Severity: normal CC: bugwatch, deloptes, slavek.banko
Priority: P5    
Version: R14.0.x [Trinity]   
Hardware: Other   
OS: Linux   
Compiler Version: TDE Version String:
Application Version: Application Name:
Bug Depends on:    
Bug Blocks: 2696    
Attachments: error (decryption failed)
signed mail when knode does not display properly
signed mail when knode displays properly
signature key exctraction patch
signature key exctraction patch
signature key exctraction patch
signature key exctraction patch

Description deloptes 2018-03-16 02:35:38 CDT
Created attachment 2829 [details]
error (decryption failed)

When trying to open signature in knode, I get an error (decryption failed).
I just wonder why it does not use gpgme the same as kmail.
Comment 1 deloptes 2018-04-05 14:53:18 CDT
Luckily I could find one such message that is displayed exactly the way it is displayed also in kmail. So it looks like it depends on the way the message is structured.

The one that works is 
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
[blah blah]
-----END PGP SIGNATURE-----

The one that doesn't is 

This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--gnknW2V3bf2fYIKGJhjNQYlTCuNkECDjz
Content-Type: multipart/mixed; boundary="rwgnE5jUACrJdnaOnspBSDfTe9mt7oV7O";
 protected-headers="v1"

[blah blah]

-----END PGP SIGNATURE-----

--gnknW2V3bf2fYIKGJhjNQYlTCuNkECDjz--
Comment 2 deloptes 2018-04-05 14:55:38 CDT
Created attachment 2830 [details]
signed mail when knode does not display properly

The signature is displayed as attachment and trying opening it causes gpg to fail
Comment 3 deloptes 2018-04-05 14:56:42 CDT
Created attachment 2831 [details]
signed mail when knode displays properly

this is one mail where it is properly displayed
Comment 4 deloptes 2018-04-05 16:15:42 CDT
Created attachment 2832 [details]
signature key exctraction patch

old style signature
Example: Signature made Sun 06 May 2001 03:49:27 PM CEST using DSA key ID 12345678

the patch will handle new style signature
Example: Signature made Thu 05 Apr 2018 10:02:50 PM CEST
               using DSA key A0CF1DC09533E5E87F54DB40F1EEB8CD9FB16A50
Comment 5 deloptes 2018-04-05 18:02:34 CDT
The other problem seems to be related to  ftp://ftp.rfc-editor.org/in-notes/rfc3156.txt
Comment 6 deloptes 2018-04-06 10:07:17 CDT
Created attachment 2833 [details]
signature key exctraction patch

previous patch extended to handle variable key size

//    gpg: Signature made Thu 05 Apr 2018 10:02:50 PM CEST
//    gpg:                using DSA key A0CF1DC09533E5E87F54DB40F1EEB8CD9FB16A50

//		gpg: Signature made Mon 02 Apr 2018 03:15:08 PM CEST
//		gpg:                using DSA key 05C82CF57AD1DA46
Comment 7 deloptes 2018-04-07 15:54:11 CDT
Created attachment 2834 [details]
signature key exctraction patch

extends the previous patch with a fix to properly display the user name in the sign/decrypt header. It prevents utf8 double encoding.

this was the last open item on knode I had for now
Comment 8 Slávek Banko 2018-05-02 15:30:52 CDT
(In reply to deloptes from comment #7)
> Created attachment 2834 [details]
> signature key exctraction patch
> 
> extends the previous patch with a fix to properly display the user name in
> the sign/decrypt header. It prevents utf8 double encoding.
> 
> this was the last open item on knode I had for now

Please, you can test whether the use of TQstring::fromLocal8Bit() will be correct instead of the currently used TQString::fromUtf8()? When "error" is the output from the gpg call, potentially it could be in another encoding => fromLocal8Bit() should be the appropriate choice.

Thank you.
Comment 9 deloptes 2018-05-02 15:50:13 CDT
Created attachment 2842 [details]
signature key exctraction patch

Hi,
yes it works with TQString::fromLocal8Bit().
I was not sure about this, although I thought it might be more appropriate.
thanks for looking into it
Comment 10 Slávek Banko 2018-05-05 12:34:25 CDT
Fixed in GIT hash f09192b2 (master) and 724a547c (r14.0.x).
Thank you for your good work!