@wireapp/core 32.0.8 → 32.1.1

package/package.json

@@ -3,7 +3,7 @@
     "./src/main/cryptography/AssetCryptography/crypto.node": "./src/main/cryptography/AssetCryptography/crypto.browser.js"
   },
   "dependencies": {
-    "@wireapp/api-client": "^20.6.8",
+    "@wireapp/api-client": "^20.6.9",
     "@wireapp/commons": "^4.4.6",
     "@wireapp/core-crypto": "0.5.2",
     "@wireapp/cryptobox": "12.8.0",
@@ -82,6 +82,6 @@
     "test": "jest",
     "watch": "tsc ---watch"
   },
-  "version": "32.0.8",
-  "gitHead": "9b39965aa991efec2376de9b40118be153d803f6"
+  "version": "32.1.1",
+  "gitHead": "0368f45ba491397a1fab4ba38f50beb85f7a58e7"
 }

package/src/main/mls/MLSService/MLSService.d.ts

@@ -14,6 +14,7 @@ export declare class MLSService {
     addUsersToExistingConversation(groupId: Uint8Array, invitee: Invitee[]): Promise<import("@wireapp/api-client/src/conversation").PostMlsMessageResponse | null>;
     configureMLSCallbacks({ groupIdFromConversationId, ...coreCryptoCallbacks }: MLSCallbacks): void;
     getKeyPackagesPayload(qualifiedUsers: QualifiedUsers[]): Promise<Invitee[]>;
+    getEpoch(groupId: Uint8Array): Promise<number>;
     newProposal(proposalType: ProposalType, args: ProposalArgs | AddProposalArgs | RemoveProposalArgs): Promise<import("@wireapp/core-crypto").ProposalBundle>;
     newExternalProposal(externalProposalType: ExternalProposalType, args: ExternalProposalArgs | ExternalRemoveProposalArgs): Promise<Uint8Array>;
     processWelcomeMessage(welcomeMessage: Uint8Array): Promise<ConversationId>;

package/src/main/mls/MLSService/MLSService.js

@@ -112,6 +112,9 @@ class MLSService {
         }, []);
         return coreCryptoKeyPackagesPayload;
     }
+    getEpoch(groupId) {
+        return this.getCoreCryptoClient().conversationEpoch(groupId);
+    }
     async newProposal(proposalType, args) {
         return this.getCoreCryptoClient().newProposal(proposalType, args);
     }

package/src/main/notification/NotificationService.d.ts

@@ -62,6 +62,13 @@ export declare class NotificationService extends EventEmitter {
     private isOutdatedEvent;
     handleNotification(notification: Notification, source: PayloadBundleSource, dryRun?: boolean): AsyncGenerator<HandledEventPayload>;
     private cleanupPayloadBundle;
+    /**
+     * Will process one event
+     * @param event The backend event to process
+     * @param source The source of the event (websocket or notication stream)
+     * @param dryRun Will not try to decrypt if true
+     * @return the decrypted payload and the raw event. Returns `undefined` when the payload is a coreCrypto-only system message
+     */
     private handleEvent;
     /**
      * ## MLS only ##

package/src/main/notification/NotificationService.js

@@ -198,7 +198,9 @@ class NotificationService extends events_1.EventEmitter {
                 }
                 try {
                     const data = yield __await(this.handleEvent(event, source, dryRun));
-                    yield yield __await(Object.assign(Object.assign({}, data), { mappedEvent: data.mappedEvent ? this.cleanupPayloadBundle(data.mappedEvent) : undefined }));
+                    if (typeof data !== 'undefined') {
+                        yield yield __await(Object.assign(Object.assign({}, data), { mappedEvent: data.mappedEvent ? this.cleanupPayloadBundle(data.mappedEvent) : undefined }));
+                    }
                 }
                 catch (error) {
                     this.logger.error(`There was an error with notification ID "${notification.id}": ${error.message}`, error);
@@ -234,6 +236,13 @@ class NotificationService extends events_1.EventEmitter {
                 return payload;
         }
     }
+    /**
+     * Will process one event
+     * @param event The backend event to process
+     * @param source The source of the event (websocket or notication stream)
+     * @param dryRun Will not try to decrypt if true
+     * @return the decrypted payload and the raw event. Returns `undefined` when the payload is a coreCrypto-only system message
+     */
     async handleEvent(event, source, dryRun = false) {
         var _a, _b;
         switch (event.type) {
@@ -251,8 +260,8 @@ class NotificationService extends events_1.EventEmitter {
                 const encryptedData = bazinga64_1.Decoder.fromBase64(event.data).asBytes;
                 const groupId = await this.getGroupIdFromConversationId((_a = event.qualified_conversation) !== null && _a !== void 0 ? _a : { id: event.conversation, domain: '' });
                 const groupIdBytes = bazinga64_1.Decoder.fromBase64(groupId).asBytes;
-                // Check if the message includes proposals
                 const { proposals, commitDelay, message } = await this.mlsService.decryptMessage(groupIdBytes, encryptedData);
+                // Check if the message includes proposals
                 if (typeof commitDelay === 'number' || proposals.length > 0) {
                     // we are dealing with a proposal, add a task to process this proposal later on
                     // Those proposals are stored inside of coreCrypto and will be handled after a timeout
@@ -262,10 +271,12 @@ class NotificationService extends events_1.EventEmitter {
                         eventTime: event.time,
                     });
                     // This is not a text message, there is nothing more to do
-                    return { event };
+                    return undefined;
                 }
                 if (!message) {
-                    throw new Error(`MLS message received from ${source} was empty`);
+                    const newEpoch = await this.mlsService.getEpoch(groupIdBytes);
+                    this.logger.log(`Received commit message for group "${groupId}". New epoch is "${newEpoch}"`);
+                    return undefined;
                 }
                 const decryptedData = protocol_messaging_1.GenericMessage.decode(message);
                 /**