diff --git a/js/conversation_controller.js b/js/conversation_controller.js index 86c228e57..559464040 100644 --- a/js/conversation_controller.js +++ b/js/conversation_controller.js @@ -213,16 +213,6 @@ ); }); }, - prepareForSend(id) { - // id is either a group id or an individual user's id - const conversation = this.get(id); - const sendOptions = {}; - const wrap = conversation - ? conversation.wrapSend.bind(conversation) - : promise => promise; - - return { wrap, sendOptions }; - }, async getAllGroupsInvolvingId(id) { const groups = await window.Signal.Data.getAllGroupsInvolvingId(id, { ConversationCollection: Whisper.ConversationCollection, diff --git a/js/models/conversations.js b/js/models/conversations.js index 1a79b1bcf..b9472f74c 100644 --- a/js/models/conversations.js +++ b/js/models/conversations.js @@ -741,17 +741,8 @@ } }, async sendVerifySyncMessage(number, state) { - // Because syncVerification sends a (null) message to the target of the verify and - // a sync message to our own devices, we need to send the accessKeys down for both - // contacts. So we merge their sendOptions. - const { sendOptions } = ConversationController.prepareForSend( - this.ourNumber, - { syncMessage: true } - ); - const options = Object.assign({}, sendOptions, {}); - const key = await textsecure.storage.protocol.loadIdentityKey(number); - return textsecure.messaging.syncVerification(number, state, key, options); + return textsecure.messaging.syncVerification(number, state, key); }, isVerified() { if (this.isPrivate()) { @@ -1460,30 +1451,6 @@ } }); }, - wrapSend(promise) { - return promise.then( - async result => { - // success - if (result) { - await this.handleMessageSendResult({ - ...result, - success: true, - }); - } - return result; - }, - async result => { - // failure - if (result) { - await this.handleMessageSendResult({ - ...result, - success: false, - }); - } - throw result; - } - ); - }, async updateAvatarOnPublicChat({ url, profileKey }) { if (!this.isPublic()) { @@ -1505,63 +1472,6 @@ ); await serverAPI.setAvatar(url, profileKey); }, - - async handleMessageSendResult({ failoverNumbers, unidentifiedDeliveries }) { - await Promise.all( - (failoverNumbers || []).map(async number => { - const conversation = ConversationController.get(number); - - if ( - conversation && - conversation.get('sealedSender') !== SEALED_SENDER.DISABLED - ) { - window.log.info( - `Setting sealedSender to DISABLED for conversation ${conversation.idForLogging()}` - ); - conversation.set({ - sealedSender: SEALED_SENDER.DISABLED, - }); - await window.Signal.Data.updateConversation( - conversation.id, - conversation.attributes, - { Conversation: Whisper.Conversation } - ); - } - }) - ); - - await Promise.all( - (unidentifiedDeliveries || []).map(async number => { - const conversation = ConversationController.get(number); - - if ( - conversation && - conversation.get('sealedSender') === SEALED_SENDER.UNKNOWN - ) { - if (conversation.get('accessKey')) { - window.log.info( - `Setting sealedSender to ENABLED for conversation ${conversation.idForLogging()}` - ); - conversation.set({ - sealedSender: SEALED_SENDER.ENABLED, - }); - } else { - window.log.info( - `Setting sealedSender to UNRESTRICTED for conversation ${conversation.idForLogging()}` - ); - conversation.set({ - sealedSender: SEALED_SENDER.UNRESTRICTED, - }); - } - await window.Signal.Data.updateConversation( - conversation.id, - conversation.attributes, - { Conversation: Whisper.Conversation } - ); - } - }) - ); - }, async updateLastMessage() { if (!this.id) { return;