From 93e226f0b78f007259f2d9418ab96139d9cd23de Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Feh=C3=A9r=20Roland?= <feherneoh@gmail.com>
Date: Tue, 30 Sep 2025 15:14:14 +0200
Subject: [PATCH] REVERT ME: debug loss of Auth key

---
 .../ImportKeysOperationCallback.java          |  4 ++++
 .../keychain/service/KeychainServiceTask.java |  3 +++
 .../ui/adapter/ImportKeysAdapter.java         | 24 ++++++++++++++++---
 .../ui/base/CryptoOperationHelper.java        |  7 +++++-
 4 files changed, 34 insertions(+), 4 deletions(-)

diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/keyimport/processing/ImportKeysOperationCallback.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/keyimport/processing/ImportKeysOperationCallback.java
index d24bf754f..39e1941b1 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/keyimport/processing/ImportKeysOperationCallback.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/keyimport/processing/ImportKeysOperationCallback.java
@@ -21,6 +21,8 @@ import org.sufficientlysecure.keychain.operations.results.ImportKeyResult;
 import org.sufficientlysecure.keychain.service.ImportKeyringParcel;
 import org.sufficientlysecure.keychain.ui.base.CryptoOperationHelper;
 
+import timber.log.Timber;
+
 public class ImportKeysOperationCallback implements
         CryptoOperationHelper.Callback<ImportKeyringParcel, ImportKeyResult> {
 
@@ -45,6 +47,7 @@ public class ImportKeysOperationCallback implements
 
     @Override
     public void onCryptoOperationSuccess(ImportKeyResult result) {
+        Timber.d("+onCryptoOperationSuccess");
         mResultListener.handleResult(result, mPosition);
     }
 
@@ -55,6 +58,7 @@ public class ImportKeysOperationCallback implements
 
     @Override
     public void onCryptoOperationError(ImportKeyResult result) {
+        Timber.d("+onCryptoOperationError");
         mResultListener.handleResult(result, mPosition);
     }
 
diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/service/KeychainServiceTask.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/service/KeychainServiceTask.java
index af82d87f9..019e15ff9 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/service/KeychainServiceTask.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/service/KeychainServiceTask.java
@@ -50,9 +50,12 @@ import org.sufficientlysecure.keychain.pgp.Progressable;
 import org.sufficientlysecure.keychain.pgp.SignEncryptParcel;
 import org.sufficientlysecure.keychain.service.input.CryptoInputParcel;
 
+import timber.log.Timber;
+
 
 public class KeychainServiceTask {
     public static KeychainServiceTask create(Activity activity) {
+        Timber.d("+create#1");
         Context context = activity.getApplicationContext();
         KeyWritableRepository keyRepository = KeyWritableRepository.create(context);
         return new KeychainServiceTask(context, keyRepository);
diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/adapter/ImportKeysAdapter.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/adapter/ImportKeysAdapter.java
index cf1a9ca4b..0329cd0f4 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/adapter/ImportKeysAdapter.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/adapter/ImportKeysAdapter.java
@@ -46,11 +46,14 @@ import org.sufficientlysecure.keychain.operations.ImportOperation;
 import org.sufficientlysecure.keychain.operations.results.ImportKeyResult;
 import org.sufficientlysecure.keychain.pgp.CanonicalizedKeyRing;
 import org.sufficientlysecure.keychain.pgp.CanonicalizedKeyRing.VerificationStatus;
+import org.sufficientlysecure.keychain.pgp.CanonicalizedPublicKey;
+import org.sufficientlysecure.keychain.pgp.exception.PgpKeyNotFoundException;
 import org.sufficientlysecure.keychain.service.ImportKeyringParcel;
 import org.sufficientlysecure.keychain.ui.base.CryptoOperationHelper;
 import org.sufficientlysecure.keychain.ui.keyview.ViewKeyActivity;
 import org.sufficientlysecure.keychain.ui.util.KeyFormattingUtils;
 import org.sufficientlysecure.keychain.ui.util.Notify;
+import org.sufficientlysecure.keychain.util.IterableIterator;
 import org.sufficientlysecure.keychain.util.ParcelableFileCache;
 import timber.log.Timber;
 
@@ -209,11 +212,17 @@ public class ImportKeysAdapter extends RecyclerView.Adapter<ImportKeysAdapter.Vi
     }
 
     private void getKey(int position, ImportKeysListEntry entry, boolean skipSave) {
+        Timber.d("+getKey position %d, entry %s, skipSave %s", position, entry.getKeyIdHex(), skipSave ? "true" : "false");
+        Timber.d("getKey => prepareKeyOperation");
         ImportKeyringParcel inputParcel = prepareKeyOperation(entry, skipSave);
         ImportKeysResultListener listener = skipSave ? this : mListener;
+        Timber.d("getKey => new ImportKeysOperationCallback");
         ImportKeysOperationCallback cb = new ImportKeysOperationCallback(listener, inputParcel, position);
+        Timber.d("getKey => new CryptoOperationHelper");
         CryptoOperationHelper opHelper = new CryptoOperationHelper<>(1, mActivity, cb, null);
+        Timber.d("getKey => opHelper.cryptoOperation");
         opHelper.cryptoOperation();
+        Timber.d("-getKey");
     }
 
     private ImportKeyringParcel prepareKeyOperation(ImportKeysListEntry entry, boolean skipSave) {
@@ -264,9 +273,18 @@ public class ImportKeysAdapter extends RecyclerView.Adapter<ImportKeysAdapter.Vi
             ArrayList<CanonicalizedKeyRing> canKeyRings = result.mCanonicalizedKeyRings;
             if (canKeyRings.size() == 1) {
                 CanonicalizedKeyRing keyRing = canKeyRings.get(0);
-                Timber.e("Key ID: " + keyRing.getMasterKeyId() +
-                        "| isRev: " + keyRing.isRevoked() + "| isExp: " + keyRing.isExpired()
-                        + "| isSec: " + keyRing.isSecure());
+                Timber.e("Key ID: %016X| isRev: " + keyRing.isRevoked() + " | isExp: " + keyRing.isExpired()
+                        + " | isSec: " + keyRing.isSecure(), keyRing.getMasterKeyId());
+                try {
+                    Timber.e("[E]: %016X", keyRing.getEncryptId());
+                } catch (PgpKeyNotFoundException ignored) {}
+                try {
+                    Timber.e("[S]: %016X", keyRing.getSigningId());
+                } catch (PgpKeyNotFoundException ignored) {}
+
+                for (CanonicalizedPublicKey val : keyRing.publicKeyIterator()) {
+                    Timber.e("KEY %016X; [%s%s%s%s]", val.getKeyId(), val.canAuthenticate() ? "A" : "", val.canCertify() ? "C" : "", val.canEncrypt() ? "E" : "", val.canSign() ? "S" : "");
+                }
 
                 ImportKeysListEntry entry = mData.get(position);
                 entry.setUpdated(result.isOkUpdated());
diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/base/CryptoOperationHelper.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/base/CryptoOperationHelper.java
index d67260b0d..e884c7444 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/base/CryptoOperationHelper.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/base/CryptoOperationHelper.java
@@ -264,6 +264,7 @@ public class CryptoOperationHelper<T extends Parcelable, S extends OperationResu
     }
 
     public void cryptoOperation(final CryptoInputParcel cryptoInput) {
+        Timber.d("+cryptoOperation#1");
         T operationInput = mCallback.createOperationInput();
         if (operationInput == null) {
             return;
@@ -320,8 +321,12 @@ public class CryptoOperationHelper<T extends Parcelable, S extends OperationResu
     }
 
     public void cryptoOperation() {
+        Timber.d("+cryptoOperation");
         operationStartTime = SystemClock.elapsedRealtime();
-        cryptoOperation(CryptoInputParcel.createCryptoInputParcel(new Date()));
+        Timber.d("cryptoOperation => CryptoInputParcel.createCryptoInputParcel");
+        CryptoInputParcel tmp = CryptoInputParcel.createCryptoInputParcel(new Date());
+        Timber.d("cryptoOperation => cryptoOperation#1");
+        cryptoOperation(tmp);
     }
 
     @UiThread