Display device information (name, id and key) in Cryptography setting screen (#1784)

CHANGES.md

@@ -6,6 +6,7 @@ Features ✨:
 
 Improvements 🙌:
  - Give user the possibility to prevent accidental call (#1869)
+ - Display device information (name, id and key) in Cryptography setting screen (#1784)
 
 Bugfix 🐛:
  - Fix invisible toolbar (Status.im theme) (#1746)

vector/src/main/java/im/vector/app/features/settings/VectorSettingsSecurityPrivacyFragment.kt

@@ -34,12 +34,6 @@ import androidx.preference.PreferenceCategory
 import androidx.preference.SwitchPreference
 import androidx.recyclerview.widget.RecyclerView
 import com.google.android.material.textfield.TextInputEditText
-import org.matrix.android.sdk.api.MatrixCallback
-import org.matrix.android.sdk.internal.crypto.crosssigning.isVerified
-import org.matrix.android.sdk.internal.crypto.model.ImportRoomKeysResult
-import org.matrix.android.sdk.internal.crypto.model.rest.DeviceInfo
-import org.matrix.android.sdk.internal.crypto.model.rest.DevicesListResponse
-import org.matrix.android.sdk.rx.SecretsSynchronisationInfo
 import im.vector.app.R
 import im.vector.app.core.di.ActiveSessionHolder
 import im.vector.app.core.dialogs.ExportKeysDialog
@@ -50,6 +44,7 @@ import im.vector.app.core.intent.getFilenameFromUri
 import im.vector.app.core.platform.SimpleTextWatcher
 import im.vector.app.core.preference.VectorPreference
 import im.vector.app.core.preference.VectorPreferenceCategory
+import im.vector.app.core.utils.copyToClipboard
 import im.vector.app.core.utils.openFileSelection
 import im.vector.app.core.utils.toast
 import im.vector.app.features.crypto.keys.KeysExporter
@@ -65,6 +60,13 @@ import im.vector.app.features.themes.ThemeUtils
 import io.reactivex.android.schedulers.AndroidSchedulers
 import io.reactivex.disposables.Disposable
 import me.gujun.android.span.span
+import org.matrix.android.sdk.api.MatrixCallback
+import org.matrix.android.sdk.api.extensions.getFingerprintHumanReadable
+import org.matrix.android.sdk.internal.crypto.crosssigning.isVerified
+import org.matrix.android.sdk.internal.crypto.model.ImportRoomKeysResult
+import org.matrix.android.sdk.internal.crypto.model.rest.DeviceInfo
+import org.matrix.android.sdk.internal.crypto.model.rest.DevicesListResponse
+import org.matrix.android.sdk.rx.SecretsSynchronisationInfo
 import org.matrix.android.sdk.rx.rx
 import javax.inject.Inject
 
@@ -85,6 +87,18 @@ class VectorSettingsSecurityPrivacyFragment @Inject constructor(
         findPreference<PreferenceCategory>(VectorPreferences.SETTINGS_CRYPTOGRAPHY_PREFERENCE_KEY)!!
     }
 
+    private val cryptoInfoDeviceNamePreference by lazy {
+        findPreference<VectorPreference>("SETTINGS_ENCRYPTION_INFORMATION_DEVICE_NAME_PREFERENCE_KEY")!!
+    }
+
+    private val cryptoInfoDeviceIdPreference by lazy {
+        findPreference<VectorPreference>("SETTINGS_ENCRYPTION_INFORMATION_DEVICE_ID_PREFERENCE_KEY")!!
+    }
+
+    private val cryptoInfoDeviceKeyPreference by lazy {
+        findPreference<VectorPreference>("SETTINGS_ENCRYPTION_INFORMATION_DEVICE_KEY_PREFERENCE_KEY")!!
+    }
+
     private val mCrossSigningStatePreference by lazy {
         findPreference<VectorPreference>(VectorPreferences.SETTINGS_ENCRYPTION_CROSS_SIGNING_PREFERENCE_KEY)!!
     }
@@ -488,58 +502,48 @@ class VectorSettingsSecurityPrivacyFragment @Inject constructor(
 
     /**
      * Build the cryptography preference section.
-     *
-     * @param aMyDeviceInfo the device info
      */
     private fun refreshCryptographyPreference(devices: List<DeviceInfo>) {
         showDeviceListPref.isEnabled = devices.isNotEmpty()
         showDeviceListPref.summary = resources.getQuantityString(R.plurals.settings_active_sessions_count, devices.size, devices.size)
-//        val userId = session.myUserId
-//        val deviceId = session.sessionParams.deviceId
 
-        // device name
+        val userId = session.myUserId
-//        if (null != aMyDeviceInfo) {
+        val deviceId = session.sessionParams.deviceId
-//            cryptoInfoDeviceNamePreference.summary = aMyDeviceInfo.displayName
+
-//
+        val aMyDeviceInfo = devices.find { it.deviceId == deviceId }
-//            cryptoInfoDeviceNamePreference.onPreferenceClickListener = Preference.OnPreferenceClickListener {
+
-//                // TODO device can be rename only from the device list screen for the moment
+        // crypto section: device name
-//                // displayDeviceRenameDialog(aMyDeviceInfo)
+        if (aMyDeviceInfo != null) {
-//                true
+            cryptoInfoDeviceNamePreference.summary = aMyDeviceInfo.displayName
-//            }
+
-//
+            cryptoInfoDeviceNamePreference.onPreferenceClickListener = Preference.OnPreferenceClickListener {
-//            cryptoInfoDeviceNamePreference.onPreferenceLongClickListener = object : VectorPreference.OnPreferenceLongClickListener {
+                copyToClipboard(requireActivity(), aMyDeviceInfo.displayName ?: "")
-//                override fun onPreferenceLongClick(preference: Preference): Boolean {
+                true
-//                    activity?.let { copyToClipboard(it, aMyDeviceInfo.displayName!!) }
+            }
-//                    return true
+        }
-//                }
+
-//            }
+        // crypto section: device ID
-//        }
+        if (!deviceId.isNullOrEmpty()) {
-//
+            cryptoInfoDeviceIdPreference.summary = deviceId
-//        // crypto section: device ID
+
-//        if (!deviceId.isNullOrEmpty()) {
+            cryptoInfoDeviceIdPreference.setOnPreferenceClickListener {
-//            cryptoInfoDeviceIdPreference.summary = deviceId
+                copyToClipboard(requireActivity(), deviceId)
-//
+                true
-//            cryptoInfoDeviceIdPreference.setOnPreferenceClickListener {
+            }
-//                activity?.let { copyToClipboard(it, deviceId) }
+        }
-//                true
+
-//            }
+        // crypto section: device key (fingerprint)
-//        }
+        val deviceInfo = session.cryptoService().getDeviceInfo(userId, deviceId)
-//
+
-//        // crypto section: device key (fingerprint)
+        val fingerprint = deviceInfo?.fingerprint()
-//        if (!deviceId.isNullOrEmpty() && userId.isNotEmpty()) {
+        if (fingerprint?.isNotEmpty() == true) {
-//            val deviceInfo = session.getDeviceInfo(userId, deviceId)
+            cryptoInfoDeviceKeyPreference.summary = deviceInfo.getFingerprintHumanReadable()
-//
+
-//            if (null != deviceInfo && !deviceInfo.fingerprint().isNullOrEmpty()) {
+            cryptoInfoDeviceKeyPreference.setOnPreferenceClickListener {
-//                cryptoInfoTextPreference.summary = deviceInfo.getFingerprintHumanReadable()
+                copyToClipboard(requireActivity(), fingerprint)
-//
+                true
-//                cryptoInfoTextPreference.setOnPreferenceClickListener {
+            }
-//                    deviceInfo.fingerprint()?.let {
+        }
-//                        copyToClipboard(requireActivity(), it)
-//                    }
-//                    true
-//                }
-//            }
-//        }
 
         sendToUnverifiedDevicesPref.isChecked = session.cryptoService().getGlobalBlacklistUnverifiedDevices()
 

vector/src/main/res/values/strings.xml

@@ -1054,7 +1054,7 @@
     <string name="encryption_information_device_name_with_warning">Public name (visible to people you communicate with)</string>
     <string name="device_name_warning">"A session's public name is visible to people you communicate with"</string>
     <string name="encryption_information_name">Public name</string>
-    <string name="encryption_information_device_id">ID</string>
+    <string name="encryption_information_device_id">Session ID</string>
     <string name="encryption_information_device_key">Session key</string>
     <string name="encryption_information_verification">Verification</string>
     <string name="encryption_information_ed25519_fingerprint">Ed25519 fingerprint</string>

vector/src/main/res/xml/vector_settings_security_privacy.xml

@@ -7,9 +7,9 @@
         android:icon="@drawable/ic_notification_privacy_warning"
         android:key="SETTINGS_CRYPTOGRAPHY_HS_ADMIN_DISABLED_E2E_DEFAULT"
         android:persistent="false"
-        tools:summary="@string/settings_hs_admin_e2e_disabled"
         app:isPreferenceVisible="false"
-        tools:isPreferenceVisible="true">
+        tools:isPreferenceVisible="true"
+        tools:summary="@string/settings_hs_admin_e2e_disabled">
 
     </im.vector.app.core.preference.VectorPreference>
 
@@ -25,18 +25,23 @@
             tools:icon="@drawable/ic_shield_trusted"
             tools:summary="@string/encryption_information_dg_xsigning_complete" />
 
+        <im.vector.app.core.preference.VectorPreference
+            android:key="SETTINGS_ENCRYPTION_INFORMATION_DEVICE_NAME_PREFERENCE_KEY"
+            android:persistent="false"
+            android:title="@string/encryption_information_device_name"
+            tools:summary="My public device name" />
 
-        <!--        <im.vector.app.core.preference.VectorPreference-->
+        <im.vector.app.core.preference.VectorPreference
-        <!--            android:key="SETTINGS_ENCRYPTION_INFORMATION_DEVICE_NAME_PREFERENCE_KEY"-->
+            android:key="SETTINGS_ENCRYPTION_INFORMATION_DEVICE_ID_PREFERENCE_KEY"
-        <!--            android:title="@string/encryption_information_device_name" />-->
+            android:persistent="false"
+            android:title="@string/encryption_information_device_id"
+            tools:summary="VZRHETBEER" />
 
-        <!--        <im.vector.app.core.preference.VectorPreference-->
+        <im.vector.app.core.preference.VectorPreference
-        <!--            android:key="SETTINGS_ENCRYPTION_INFORMATION_DEVICE_ID_PREFERENCE_KEY"-->
+            android:key="SETTINGS_ENCRYPTION_INFORMATION_DEVICE_KEY_PREFERENCE_KEY"
-        <!--            android:title="@string/encryption_information_device_id" />-->
+            android:persistent="false"
-
+            android:title="@string/encryption_information_device_key"
-        <!--        <im.vector.app.core.preference.VectorPreference-->
+            tools:summary="3To0 8c/K VRJd 4Njb DUgv 6r8A SNp9 ETZt pMwU CpE4 XJE" />
-        <!--            android:key="SETTINGS_ENCRYPTION_INFORMATION_DEVICE_KEY_PREFERENCE_KEY"-->
-        <!--            android:title="@string/encryption_information_device_key" />-->
 
         <im.vector.app.core.preference.VectorSwitchPreference
             android:enabled="true"