improve logging when PGP decryption fails

Daniel Gultsch created 2 years ago

Change summary

src/main/java/eu/siacs/conversations/crypto/PgpDecryptionService.java    | 4 
src/main/java/eu/siacs/conversations/services/XmppConnectionService.java | 5 
2 files changed, 6 insertions(+), 3 deletions(-)

Detailed changes

src/main/java/eu/siacs/conversations/crypto/PgpDecryptionService.java

@@ -156,7 +156,8 @@ public class PgpDecryptionService {
 									&& manager.getAutoAcceptFileSize() > 0) {
 								manager.createNewDownloadConnection(message);
 							}
-						} catch (IOException e) {
+						} catch (final IOException e) {
+							Log.d(Config.LOGTAG,"decryption failed", e);
 							message.setEncryption(Message.ENCRYPTION_DECRYPTION_FAILED);
 						}
 						mXmppConnectionService.updateMessage(message);
@@ -170,6 +171,7 @@ public class PgpDecryptionService {
 						}
 						break;
 					case OpenPgpApi.RESULT_CODE_ERROR:
+						Log.d(Config.LOGTAG,"decryption failed (api error)");
 						message.setEncryption(Message.ENCRYPTION_DECRYPTION_FAILED);
 						mXmppConnectionService.updateMessage(message);
 						break;

src/main/java/eu/siacs/conversations/services/XmppConnectionService.java

@@ -1259,7 +1259,7 @@ public class XmppConnectionService extends Service {
         if (Config.supportOpenPgp()) {
             this.pgpServiceConnection = new OpenPgpServiceConnection(this, "org.sufficientlysecure.keychain", new OpenPgpServiceConnection.OnBound() {
                 @Override
-                public void onBound(IOpenPgpService2 service) {
+                public void onBound(final IOpenPgpService2 service) {
                     for (Account account : accounts) {
                         final PgpDecryptionService pgp = account.getPgpDecryptionService();
                         if (pgp != null) {
@@ -1269,7 +1269,8 @@ public class XmppConnectionService extends Service {
                 }
 
                 @Override
-                public void onError(Exception e) {
+                public void onError(final Exception exception) {
+                    Log.e(Config.LOGTAG,"could not bind to OpenKeyChain", exception);
                 }
             });
             this.pgpServiceConnection.bindToService();