@wireapp/core 30.3.0 → 30.4.0

package/CHANGELOG.md

@@ -3,6 +3,17 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [30.4.0](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/compare/@wireapp/core@30.3.0...@wireapp/core@30.4.0) (2022-09-07)
+
+
+### Features
+
+* Update getAllParticipantsClients to use better endpoints without a hack ([#4379](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/issues/4379)) ([f38258d](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/commit/f38258db39e81c4b517126792aa6c605b1ea51c5))
+
+
+
+
+
 # [30.3.0](https://github.com/wireapp/wire-web-packages/tree/main/packages/core/compare/@wireapp/core@30.2.0...@wireapp/core@30.3.0) (2022-09-07)
 
 

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.1.0",
+    "@wireapp/api-client": "20.2.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.3.0",
-  "gitHead": "0a4d4b6057006da93a731115ab02c9753b155127"
+  "version": "30.4.0",
+  "gitHead": "9df4ed23ee62258dac598e16e86ef037ee143f6e"
 }

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

@@ -92,10 +92,6 @@ 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.
-     *
      * @param {string} conversationId
      * @param {string} conversationDomain? - If given will send the message to the new qualified endpoint
      */

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

@@ -63,11 +63,10 @@ class ConversationService {
          * yourself in the list of users if you want to sync a message also to your
          * other clients.
          */
-        return (conversation.members.others
+        return conversation.members.others
             .filter(member => !!member.qualified_id)
             .map(member => member.qualified_id)
-            // TODO(Federation): Use 'domain' from 'conversation.members.self' when backend has it implemented
-            .concat({ domain: this.apiClient.context.domain, id: conversation.members.self.id }));
+            .concat(conversation.members.self.qualified_id);
     }
     /**
      * Will generate a prekey bundle for specific users.
@@ -515,37 +514,18 @@ 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.
-     *
      * @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,
-                });
-            }
-        });
+    async getAllParticipantsClients(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 = {};
+        Object.entries(allClients.qualified_user_map).map(([domain, userClientMap]) => Object.entries(userClientMap).map(async ([userId, clients]) => {
+            qualifiedUserClients[domain] || (qualifiedUserClients[domain] = {});
+            qualifiedUserClients[domain][userId] = clients.map(client => client.id);
+        }));
+        return qualifiedUserClients;
     }
     async deleteMessageLocal(conversationId, messageIdToHide, sendAsProtobuf, conversationDomain) {
         const messageId = MessageBuilder_1.MessageBuilder.createId();