support multiple pub keys per recipient

FlowCrypt/src/main/java/com/flowcrypt/email/api/email/EmailUtil.kt

@@ -687,7 +687,7 @@ class EmailUtil {
       if (outgoingMsgInfo.encryptionType === MessageEncryptionType.ENCRYPTED) {
         val recipients = outgoingMsgInfo.getAllRecipients().toMutableList()
         pubKeys = mutableListOf()
-        pubKeys.addAll(SecurityUtils.getRecipientsPubKeys(context, recipients))
+        pubKeys.addAll(SecurityUtils.getRecipientsUsablePubKeys(context, recipients))
         pubKeys.addAll(senderPgpKeyDetailsList.map { it.publicKey })
         prvKeys = listOf(
           senderPgpKeyDetailsList.firstOrNull()?.privateKey

FlowCrypt/src/main/java/com/flowcrypt/email/database/FlowCryptRoomDatabase.kt

@@ -526,18 +526,11 @@ abstract class FlowCryptRoomDatabase : RoomDatabase() {
         database.execSQL("CREATE TEMP TABLE IF NOT EXISTS contacts_temp AS SELECT * FROM contacts;")
 
         //create `recipients` table
-        database.execSQL(
-          """CREATE TABLE IF NOT EXISTS `recipients`
-           (`_id` INTEGER PRIMARY KEY AUTOINCREMENT, `email` TEXT NOT NULL,
-            `name` TEXT DEFAULT NULL, `last_use` INTEGER NOT NULL DEFAULT 0)""".trimMargin()
-        )
+        database.execSQL("CREATE TABLE IF NOT EXISTS `recipients` (`_id` INTEGER PRIMARY KEY AUTOINCREMENT, `email` TEXT NOT NULL, `name` TEXT DEFAULT NULL, `last_use` INTEGER NOT NULL DEFAULT 0)")
         database.execSQL("CREATE INDEX IF NOT EXISTS `name_in_recipients` ON `recipients` (`name`)")
         database.execSQL("CREATE INDEX IF NOT EXISTS `last_use_in_recipients` ON `recipients` (`last_use`)")
         database.execSQL("CREATE UNIQUE INDEX IF NOT EXISTS `email_in_recipients` ON `recipients` (`email`)")
-        database.execSQL(
-          "INSERT INTO recipients(email, name, last_use)" +
-              " SELECT email, name, last_use FROM contacts_temp"
-        )
+        database.execSQL("INSERT INTO recipients(email, name, last_use) SELECT email, name, last_use FROM contacts_temp")
 
         //create `public_keys` table
         database.execSQL("CREATE TABLE IF NOT EXISTS `public_keys` (`_id` INTEGER PRIMARY KEY AUTOINCREMENT, `recipient` TEXT NOT NULL, `fingerprint` TEXT NOT NULL, `public_key` BLOB NOT NULL, FOREIGN KEY(`recipient`) REFERENCES `recipients`(`email`) ON UPDATE NO ACTION ON DELETE CASCADE )")

FlowCrypt/src/main/java/com/flowcrypt/email/database/dao/PubKeyDao.kt

@@ -18,10 +18,10 @@ import kotlinx.coroutines.flow.Flow
  */
 @Dao
 interface PubKeyDao : BaseDao<PublicKeyEntity> {
-  @Query("SELECT * FROM public_keys")
+  @Query(SELECT_ALL_PUB_KEYS)
   suspend fun getAllPublicKeys(): List<PublicKeyEntity>
 
-  @Query("SELECT * FROM public_keys")
+  @Query(SELECT_ALL_PUB_KEYS)
   fun getAllPublicKeysFlow(): Flow<List<PublicKeyEntity>>
 
   @Query("SELECT * FROM public_keys WHERE recipient = :recipient")
@@ -41,4 +41,8 @@ interface PubKeyDao : BaseDao<PublicKeyEntity> {
 
   @Query("SELECT * FROM public_keys WHERE _id = :id")
   fun getPublicKeyByIdFlow(id: Long): Flow<PublicKeyEntity?>
+
+  companion object {
+    private const val SELECT_ALL_PUB_KEYS = "SELECT * FROM public_keys"
+  }
 }

FlowCrypt/src/main/java/com/flowcrypt/email/jetpack/viewmodel/RecipientsViewModel.kt

@@ -137,7 +137,7 @@ class RecipientsViewModel(application: Application) : AccountViewModel(applicati
     }
   }
 
-  fun deleteContact(recipientEntity: RecipientEntity) {
+  fun deleteRecipient(recipientEntity: RecipientEntity) {
     viewModelScope.launch {
       roomDatabase.recipientDao().deleteSuspend(recipientEntity)
     }
@@ -220,7 +220,7 @@ class RecipientsViewModel(application: Application) : AccountViewModel(applicati
     }
   }
 
-  fun filterContacts(searchPattern: String?) {
+  fun filterContacts(searchPattern: String) {
     searchPatternLiveData.value = searchPattern
   }
 

FlowCrypt/src/main/java/com/flowcrypt/email/jetpack/workmanager/ForwardedAttachmentsDownloaderWorker.kt

@@ -231,7 +231,7 @@ class ForwardedAttachmentsDownloaderWorker(context: Context, params: WorkerParam
       val senderEmail = EmailUtil.getFirstAddressString(msgEntity.from)
       val recipients = msgEntity.allRecipients.toMutableList()
       pubKeys = mutableListOf()
-      pubKeys.addAll(SecurityUtils.getRecipientsPubKeys(applicationContext, recipients))
+      pubKeys.addAll(SecurityUtils.getRecipientsUsablePubKeys(applicationContext, recipients))
       pubKeys.addAll(
         SecurityUtils.getSenderPgpKeyDetailsList(applicationContext, account, senderEmail)
           .map { it.publicKey }

FlowCrypt/src/main/java/com/flowcrypt/email/security/SecurityUtils.kt

@@ -114,7 +114,7 @@ class SecurityUtils {
      * @throws NoKeyAvailableException
      */
     @JvmStatic
-    fun getRecipientsPubKeys(context: Context, emails: MutableList<String>): List<String> {
+    fun getRecipientsUsablePubKeys(context: Context, emails: MutableList<String>): List<String> {
       val publicKeys = mutableListOf<String>()
       val recipientsWithPubKeys = FlowCryptRoomDatabase.getDatabase(context).recipientDao()
         .getRecipientsWithPubKeysByEmails(emails)

FlowCrypt/src/main/java/com/flowcrypt/email/service/PrepareOutgoingMessagesJobIntentService.kt

@@ -263,7 +263,7 @@ class PrepareOutgoingMessagesJobIntentService : JobIntentService() {
       val senderEmail = outgoingMsgInfo.from
       val recipients = outgoingMsgInfo.getAllRecipients().toMutableList()
       pubKeys = mutableListOf()
-      pubKeys.addAll(SecurityUtils.getRecipientsPubKeys(applicationContext, recipients))
+      pubKeys.addAll(SecurityUtils.getRecipientsUsablePubKeys(applicationContext, recipients))
       pubKeys.addAll(
         SecurityUtils.getSenderPgpKeyDetailsList(applicationContext, accountEntity, senderEmail)
           .map { it.publicKey }

FlowCrypt/src/main/java/com/flowcrypt/email/ui/activity/fragment/RecipientsListFragment.kt

@@ -40,7 +40,7 @@ class RecipientsListFragment : BaseFragment(), ListProgressBehaviour {
       true,
       object : RecipientsRecyclerViewAdapter.OnRecipientActionsListener {
         override fun onDeleteRecipient(recipientEntity: RecipientEntity) {
-          recipientsViewModel.deleteContact(recipientEntity)
+          recipientsViewModel.deleteRecipient(recipientEntity)
           Toast.makeText(
             context, getString(R.string.the_contact_was_deleted, recipientEntity.email),
             Toast.LENGTH_SHORT

FlowCrypt/src/main/java/com/flowcrypt/email/ui/activity/fragment/base/CreateMessageFragment.kt

@@ -149,7 +149,6 @@ class CreateMessageFragment : BaseSyncFragment(), View.OnFocusChangeListener,
   private var imageButtonAdditionalRecipientsVisibility: View? = null
   private var iBShowQuotedText: View? = null
 
-  private var isRecipientsUpdateEnabled = true
   private var isUpdateToCompleted = true
   private var isUpdateCcCompleted = true
   private var isUpdateBccCompleted = true
@@ -862,12 +861,8 @@ class CreateMessageFragment : BaseSyncFragment(), View.OnFocusChangeListener,
       if (hasFocus) {
         recipients?.clear()
       } else {
-        if (isRecipientsUpdateEnabled) {
-          if (isAdded) {
-            fetchDetailsAboutRecipients(type)
-          }
-        } else {
-          progressBar?.visibility = View.INVISIBLE
+        if (isAdded) {
+          fetchDetailsAboutRecipients(type)
         }
       }
     }
@@ -1498,8 +1493,6 @@ class CreateMessageFragment : BaseSyncFragment(), View.OnFocusChangeListener,
    */
   private fun sendMsg() {
     dismissCurrentSnackBar()
-
-    isRecipientsUpdateEnabled = false
     onMsgSendListener.sendMsg(getOutgoingMsgInfo())
   }