-
-
Notifications
You must be signed in to change notification settings - Fork 624
Open
Labels
A-E2EES-MajorSeverely degrades major functionality or product features, with no satisfactory workaroundSeverely degrades major functionality or product features, with no satisfactory workaroundT-Defect
Description
When working on element-hq/element-web#28278 to migrate to use cryptoApi.isEncryptionEnabledInRoom
, the test decryption-failure-messages.spec.ts > decryption failure messages
was in failure when https://github.com/element-hq/element-web/blob/de820e11fc2bc20eb608055205e1e44d11f2c740/playwright/e2e/crypto/decryption-failure-messages.spec.ts#L70-L72 is tested.
It's failing because the room is taking too much time to be displayed because cryptoApi. isEncryptionEnabledInRoom
is taking up to 4sec to respond in this case.
this.olmMachine.getRoomSettings(new RustSdkCryptoJs.RoomId(roomId));
call is causing this slowness:
matrix-js-sdk/src/rust-crypto/rust-crypto.ts
Lines 383 to 391 in 69647a3
/** | |
* Implementation of {@link CryptoApi#isEncryptionEnabledInRoom}. | |
*/ | |
public async isEncryptionEnabledInRoom(roomId: string): Promise<boolean> { | |
const roomSettings: RustSdkCryptoJs.RoomSettings | undefined = await this.olmMachine.getRoomSettings( | |
new RustSdkCryptoJs.RoomId(roomId), | |
); | |
return Boolean(roomSettings?.algorithm); | |
} |
Metadata
Metadata
Assignees
Labels
A-E2EES-MajorSeverely degrades major functionality or product features, with no satisfactory workaroundSeverely degrades major functionality or product features, with no satisfactory workaroundT-Defect