@wireapp/core 30.5.0 → 30.6.0

package/CHANGELOG.md

@@ -3,6 +3,38 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [30.6.0](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/compare/@wireapp/core@30.5.2...@wireapp/core@30.6.0) (2022-09-13)
+
+
+### Bug Fixes
+
+* restore initial getAllParticipantsClients ([#4388](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/issues/4388)) ([614b908](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/commit/614b908e2a2410c4d6cb1510bd66d03d0d609c84))
+
+
+### Features
+
+* add fetchAllParticipantsClients method for MLS conversations ([#4389](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/issues/4389)) ([e3e0a3d](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/commit/e3e0a3d00ce52d3ff9f2637f8e0ca655551a23fd))
+
+
+
+
+
+## [30.5.2](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/compare/@wireapp/core@30.5.1...@wireapp/core@30.5.2) (2022-09-12)
+
+**Note:** Version bump only for package @wireapp/core
+
+
+
+
+
+## [30.5.1](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/compare/@wireapp/core@30.5.0...@wireapp/core@30.5.1) (2022-09-12)
+
+**Note:** Version bump only for package @wireapp/core
+
+
+
+
+
 # [30.5.0](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/compare/@wireapp/core@30.4.1...@wireapp/core@30.5.0) (2022-09-09)
 
 

package/package.json

@@ -7,7 +7,7 @@
     "@otak/core-crypto": "0.3.0-es2017",
     "@types/long": "4.0.1",
     "@types/node": "~14",
-    "@wireapp/api-client": "20.2.1",
+    "@wireapp/api-client": "20.3.0",
     "@wireapp/commons": "4.3.0",
     "@wireapp/cryptobox": "12.8.0",
     "@wireapp/store-engine-dexie": "1.6.10",
@@ -77,6 +77,6 @@
     "test:node": "nyc jasmine --config=jasmine.json",
     "watch": "tsc ---watch"
   },
-  "version": "30.5.0",
-  "gitHead": "a2a55f074ebfb3fe5e3a32df1153a9c1ae7091dd"
+  "version": "30.6.0",
+  "gitHead": "d440686b77284f831240e66ee649dac346fe2ccc"
 }

package/src/main/conversation/ConversationService/ConversationService.d.ts

@@ -92,10 +92,22 @@ export declare class ConversationService {
     })>;
     /**
      * Get a fresh list from backend of clients for all the participants of the conversation.
+     * This is a hacky way of getting all the clients for a conversation.
+     * The idea is to send an empty message to the backend to absolutely no users and let backend reply with a mismatch error.
+     * We then get the missing members in the mismatch, that is our fresh list of participants' clients.
+     *
+     * @deprecated
      * @param {string} conversationId
      * @param {string} conversationDomain? - If given will send the message to the new qualified endpoint
      */
     getAllParticipantsClients(conversationId: string, conversationDomain?: string): Promise<UserClients | QualifiedUserClients>;
+    /**
+     * Get a fresh list from backend of clients for all the participants of the conversation.
+     * @fixme there are some case where this method is not enough to detect removed devices
+     * @param {string} conversationId
+     * @param {string} conversationDomain? - If given will send the message to the new qualified endpoint
+     */
+    fetchAllParticipantsClients(conversationId: string, conversationDomain?: string): Promise<UserClients | QualifiedUserClients>;
     deleteMessageLocal(conversationId: string, messageIdToHide: string, sendAsProtobuf?: boolean, conversationDomain?: string): Promise<HideMessage>;
     deleteMessageEveryone(conversationId: string, messageIdToDelete: string, userIds?: string[] | QualifiedId[] | UserClients | QualifiedUserClients, sendAsProtobuf?: boolean, conversationDomain?: string, callbacks?: MessageSendingCallbacks): Promise<DeleteMessage>;
     leaveConversation(conversationId: QualifiedId): Promise<ConversationMemberLeaveEvent>;

package/src/main/conversation/ConversationService/ConversationService.js

@@ -510,10 +510,46 @@ class ConversationService {
     }
     /**
      * Get a fresh list from backend of clients for all the participants of the conversation.
+     * This is a hacky way of getting all the clients for a conversation.
+     * The idea is to send an empty message to the backend to absolutely no users and let backend reply with a mismatch error.
+     * We then get the missing members in the mismatch, that is our fresh list of participants' clients.
+     *
+     * @deprecated
+     * @param {string} conversationId
+     * @param {string} conversationDomain? - If given will send the message to the new qualified endpoint
+     */
+    getAllParticipantsClients(conversationId, conversationDomain) {
+        const sendingClientId = this.apiClient.validatedClientId;
+        const recipients = {};
+        const text = new Uint8Array();
+        return new Promise(async (resolve) => {
+            const onClientMismatch = (mismatch) => {
+                resolve(mismatch.missing);
+                // When the mismatch happens, we ask the messageService to cancel the sending
+                return false;
+            };
+            if (conversationDomain && this.config.useQualifiedIds) {
+                await this.messageService.sendFederatedMessage(sendingClientId, recipients, text, {
+                    conversationId: { id: conversationId, domain: conversationDomain },
+                    onClientMismatch,
+                    reportMissing: true,
+                });
+            }
+            else {
+                await this.messageService.sendMessage(sendingClientId, recipients, text, {
+                    conversationId,
+                    onClientMismatch,
+                });
+            }
+        });
+    }
+    /**
+     * Get a fresh list from backend of clients for all the participants of the conversation.
+     * @fixme there are some case where this method is not enough to detect removed devices
      * @param {string} conversationId
      * @param {string} conversationDomain? - If given will send the message to the new qualified endpoint
      */
-    async getAllParticipantsClients(conversationId, conversationDomain) {
+    async fetchAllParticipantsClients(conversationId, conversationDomain) {
         const qualifiedMembers = await this.getConversationQualifiedMembers(conversationDomain ? { id: conversationId, domain: conversationDomain } : conversationId);
         const allClients = await this.apiClient.api.user.postListClients({ qualified_users: qualifiedMembers });
         const qualifiedUserClients = {};

package/src/main/cryptography/CryptographyDatabaseRepository.js

@@ -35,7 +35,9 @@ class CryptographyDatabaseRepository {
         this.storeEngine = storeEngine;
     }
     deleteStores() {
-        return Promise.all(Object.keys(CryptographyDatabaseRepository.STORES).map(store => this.storeEngine.deleteAll(store)));
+        return Promise.all(
+        //make sure we use enum's lowercase values, not uppercase keys
+        Object.values(CryptographyDatabaseRepository.STORES).map(store => this.storeEngine.deleteAll(store)));
     }
 }
 exports.CryptographyDatabaseRepository = CryptographyDatabaseRepository;