From d813e6d9321e14c96a409a1de73b4cc9be1b8266 Mon Sep 17 00:00:00 2001 From: Bruno Windels Date: Fri, 28 Aug 2020 14:36:00 +0200 Subject: [PATCH] store encryption event content rather than just flag in room summary --- src/matrix/room/Room.js | 4 ++++ src/matrix/room/RoomSummary.js | 14 +++++++++++--- 2 files changed, 15 insertions(+), 3 deletions(-) diff --git a/src/matrix/room/Room.js b/src/matrix/room/Room.js index 27031203..c7b5e736 100644 --- a/src/matrix/room/Room.js +++ b/src/matrix/room/Room.js @@ -256,6 +256,10 @@ export class Room extends EventEmitter { return !!(tags && tags['m.lowpriority']); } + get isEncrypted() { + return !!this._summary.encryption; + } + async _getLastEventId() { const lastKey = this._syncWriter.lastMessageKey; if (lastKey) { diff --git a/src/matrix/room/RoomSummary.js b/src/matrix/room/RoomSummary.js index e910c4bc..803aff49 100644 --- a/src/matrix/room/RoomSummary.js +++ b/src/matrix/room/RoomSummary.js @@ -14,6 +14,8 @@ See the License for the specific language governing permissions and limitations under the License. */ +import {MEGOLM_ALGORITHM} from "../e2ee/common.js"; + function applySyncResponse(data, roomResponse, membership, isInitialSync, isTimelineOpen, ownUserId) { if (roomResponse.summary) { data = updateSummary(data, roomResponse.summary); @@ -68,9 +70,10 @@ function processRoomAccountData(data, event) { function processStateEvent(data, event) { if (event.type === "m.room.encryption") { - if (!data.isEncrypted) { + const algorithm = event.content?.algorithm; + if (!data.encryption && algorithm === MEGOLM_ALGORITHM) { data = data.cloneIfNeeded(); - data.isEncrypted = true; + data.encryption = event.content; } } else if (event.type === "m.room.name") { const newName = event.content?.name; @@ -136,7 +139,7 @@ class SummaryData { this.lastMessageBody = copy ? copy.lastMessageBody : null; this.lastMessageTimestamp = copy ? copy.lastMessageTimestamp : null; this.isUnread = copy ? copy.isUnread : false; - this.isEncrypted = copy ? copy.isEncrypted : false; + this.encryption = copy ? copy.encryption : null; this.isDirectMessage = copy ? copy.isDirectMessage : false; this.membership = copy ? copy.membership : null; this.inviteCount = copy ? copy.inviteCount : 0; @@ -190,6 +193,11 @@ export class RoomSummary { return this._data.heroes; } + get encryption() { + return this._data.encryption; + } + + // whether the room name should be determined with Heroes get needsHeroes() { return needsHeroes(this._data); }