@unwanted/matrix-sdk-mini 34.12.0 → 34.13.0

package/src/rust-crypto/OutgoingRequestsManager.ts

@@ -1,143 +0,0 @@
-/*
-Copyright 2023 The Matrix.org Foundation C.I.C.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-import { OlmMachine } from "@matrix-org/matrix-sdk-crypto-wasm";
-
-import { OutgoingRequest, OutgoingRequestProcessor } from "./OutgoingRequestProcessor.ts";
-import { Logger } from "../logger.ts";
-import { defer, IDeferred, logDuration } from "../utils.ts";
-
-/**
- * OutgoingRequestsManager: responsible for processing outgoing requests from the OlmMachine.
- * Ensure that only one loop is going on at once, and that the requests are processed in order.
- */
-export class OutgoingRequestsManager {
-    /** whether {@link stop} has been called */
-    private stopped = false;
-
-    /** whether {@link outgoingRequestLoop} is currently running */
-    private outgoingRequestLoopRunning = false;
-
-    /**
-     * If there are additional calls to doProcessOutgoingRequests() while there is a current call running
-     * we need to remember in order to call `doProcessOutgoingRequests` again (as there could be new requests).
-     *
-     * If this is defined, it is an indication that we need to do another iteration; in this case the deferred
-     * will resolve once that next iteration completes. If it is undefined, there have been no new calls
-     * to `doProcessOutgoingRequests` since the current iteration started.
-     */
-    private nextLoopDeferred?: IDeferred<void>;
-
-    public constructor(
-        private readonly logger: Logger,
-        private readonly olmMachine: OlmMachine,
-        public readonly outgoingRequestProcessor: OutgoingRequestProcessor,
-    ) {}
-
-    /**
-     * Shut down as soon as possible the current loop of outgoing requests processing.
-     */
-    public stop(): void {
-        this.stopped = true;
-    }
-
-    /**
-     * Process the OutgoingRequests from the OlmMachine.
-     *
-     * This should be called at the end of each sync, to process any OlmMachine OutgoingRequests created by the rust sdk.
-     * In some cases if OutgoingRequests need to be sent immediately, this can be called directly.
-     *
-     * Calls to doProcessOutgoingRequests() are processed synchronously, one after the other, in order.
-     * If doProcessOutgoingRequests() is called while another call is still being processed, it will be queued.
-     * Multiple calls to doProcessOutgoingRequests() when a call is already processing will be batched together.
-     */
-    public doProcessOutgoingRequests(): Promise<void> {
-        // Flag that we need at least one more iteration of the loop.
-        //
-        // It is important that we do this even if the loop is currently running. There is potential for a race whereby
-        // a request is added to the queue *after* `OlmMachine.outgoingRequests` checks the queue, but *before* it
-        // returns. In such a case, the item could sit there unnoticed for some time.
-        //
-        // In order to circumvent the race, we set a flag which tells the loop to go round once again even if the
-        // queue appears to be empty.
-        if (!this.nextLoopDeferred) {
-            this.nextLoopDeferred = defer();
-        }
-
-        // ... and wait for it to complete.
-        const result = this.nextLoopDeferred.promise;
-
-        // set the loop going if it is not already.
-        if (!this.outgoingRequestLoopRunning) {
-            this.outgoingRequestLoop().catch((e) => {
-                // this should not happen; outgoingRequestLoop should return any errors via `nextLoopDeferred`.
-                /* istanbul ignore next */
-                this.logger.error("Uncaught error in outgoing request loop", e);
-            });
-        }
-        return result;
-    }
-
-    private async outgoingRequestLoop(): Promise<void> {
-        /* istanbul ignore if */
-        if (this.outgoingRequestLoopRunning) {
-            throw new Error("Cannot run two outgoing request loops");
-        }
-        this.outgoingRequestLoopRunning = true;
-        try {
-            while (!this.stopped && this.nextLoopDeferred) {
-                const deferred = this.nextLoopDeferred;
-
-                // reset `nextLoopDeferred` so that any future calls to `doProcessOutgoingRequests` are queued
-                // for another additional iteration.
-                this.nextLoopDeferred = undefined;
-
-                // make the requests and feed the results back to the `nextLoopDeferred`
-                await this.processOutgoingRequests().then(deferred.resolve, deferred.reject);
-            }
-        } finally {
-            this.outgoingRequestLoopRunning = false;
-        }
-
-        if (this.nextLoopDeferred) {
-            // the loop was stopped, but there was a call to `doProcessOutgoingRequests`. Make sure that
-            // we reject the promise in case anything is waiting for it.
-            this.nextLoopDeferred.reject(new Error("OutgoingRequestsManager was stopped"));
-        }
-    }
-
-    /**
-     * Make a single request to `olmMachine.outgoingRequests` and do the corresponding requests.
-     */
-    private async processOutgoingRequests(): Promise<void> {
-        if (this.stopped) return;
-
-        const outgoingRequests: OutgoingRequest[] = await this.olmMachine.outgoingRequests();
-
-        for (const request of outgoingRequests) {
-            if (this.stopped) return;
-            try {
-                await logDuration(this.logger, `Make outgoing request ${request.type}`, async () => {
-                    await this.outgoingRequestProcessor.makeOutgoingRequest(request);
-                });
-            } catch (e) {
-                // as part of the loop we silently ignore errors, but log them.
-                // The rust sdk will retry the request later as it won't have been marked as sent.
-                this.logger.error(`Failed to process outgoing request ${request.type}: ${e}`);
-            }
-        }
-    }
-}

package/src/rust-crypto/PerSessionKeyBackupDownloader.ts

@@ -1,501 +0,0 @@
-/*
-Copyright 2023 - 2024 The Matrix.org Foundation C.I.C.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-import * as RustSdkCryptoJs from "@matrix-org/matrix-sdk-crypto-wasm";
-import { OlmMachine } from "@matrix-org/matrix-sdk-crypto-wasm";
-
-import { Curve25519AuthData, KeyBackupInfo, KeyBackupSession } from "../crypto-api/keybackup.ts";
-import { CryptoEvent } from "../crypto-api/index.ts";
-import { Logger } from "../logger.ts";
-import { ClientPrefix, IHttpOpts, MatrixError, MatrixHttpApi, Method } from "../http-api/index.ts";
-import { RustBackupManager } from "./backup.ts";
-import { encodeUri, sleep } from "../utils.ts";
-import { BackupDecryptor } from "../common-crypto/CryptoBackend.ts";
-
-// The minimum time to wait between two retries in case of errors. To avoid hammering the server.
-const KEY_BACKUP_BACKOFF = 5000; // ms
-
-/**
- * Enumerates the different kind of errors that can occurs when downloading and importing a key from backup.
- */
-enum KeyDownloadErrorCode {
-    /** The requested key is not in the backup. */
-    MISSING_DECRYPTION_KEY = "MISSING_DECRYPTION_KEY",
-    /** A network error occurred while trying to download the key from backup. */
-    NETWORK_ERROR = "NETWORK_ERROR",
-    /** The loop has been stopped. */
-    STOPPED = "STOPPED",
-}
-
-class KeyDownloadError extends Error {
-    public constructor(public readonly code: KeyDownloadErrorCode) {
-        super(`Failed to get key from backup: ${code}`);
-        this.name = "KeyDownloadError";
-    }
-}
-
-class KeyDownloadRateLimitError extends Error {
-    public constructor(public readonly retryMillis: number) {
-        super(`Failed to get key from backup: rate limited`);
-        this.name = "KeyDownloadRateLimitError";
-    }
-}
-
-/** Details of a megolm session whose key we are trying to fetch. */
-type SessionInfo = { roomId: string; megolmSessionId: string };
-
-/** Holds the current backup decryptor and version that should be used.
- *
- * This is intended to be used as an immutable object (a new instance should be created if the configuration changes),
- * and some of the logic relies on that, so the properties are marked as `readonly`.
- */
-type Configuration = {
-    readonly backupVersion: string;
-    readonly decryptor: BackupDecryptor;
-};
-
-/**
- * Used when an 'unable to decrypt' error occurs. It attempts to download the key from the backup.
- *
- * The current backup API lacks pagination, which can lead to lengthy key retrieval times for large histories (several 10s of minutes).
- * To mitigate this, keys are downloaded on demand as decryption errors occurs.
- * While this approach may result in numerous requests, it improves user experience by reducing wait times for message decryption.
- *
- * The PerSessionKeyBackupDownloader is resistant to backup configuration changes: it will automatically resume querying when
- * the backup is configured correctly.
- */
-export class PerSessionKeyBackupDownloader {
-    private stopped = false;
-
-    /**
-     * The version and decryption key to use with current backup if all set up correctly.
-     *
-     * Will not be set unless `hasConfigurationProblem` is `false`.
-     */
-    private configuration: Configuration | null = null;
-
-    /** We remember when a session was requested and not found in backup to avoid query again too soon.
-     * Map of session_id to timestamp */
-    private sessionLastCheckAttemptedTime: Map<string, number> = new Map();
-
-    /** The logger to use */
-    private readonly logger: Logger;
-
-    /** Whether the download loop is running. */
-    private downloadLoopRunning = false;
-
-    /** The list of requests that are queued. */
-    private queuedRequests: SessionInfo[] = [];
-
-    /** Remembers if we have a configuration problem. */
-    private hasConfigurationProblem = false;
-
-    /** The current server backup version check promise. To avoid doing a server call if one is in flight. */
-    private currentBackupVersionCheck: Promise<Configuration | null> | null = null;
-
-    /**
-     * Creates a new instance of PerSessionKeyBackupDownloader.
-     *
-     * @param backupManager - The backup manager to use.
-     * @param olmMachine - The olm machine to use.
-     * @param http - The http instance to use.
-     * @param logger - The logger to use.
-     */
-    public constructor(
-        logger: Logger,
-        private readonly olmMachine: OlmMachine,
-        private readonly http: MatrixHttpApi<IHttpOpts & { onlyData: true }>,
-        private readonly backupManager: RustBackupManager,
-    ) {
-        this.logger = logger.getChild("[PerSessionKeyBackupDownloader]");
-
-        backupManager.on(CryptoEvent.KeyBackupStatus, this.onBackupStatusChanged);
-        backupManager.on(CryptoEvent.KeyBackupFailed, this.onBackupStatusChanged);
-        backupManager.on(CryptoEvent.KeyBackupDecryptionKeyCached, this.onBackupStatusChanged);
-    }
-
-    /**
-     * Check if key download is successfully configured and active.
-     *
-     * @return `true` if key download is correctly configured and active; otherwise `false`.
-     */
-    public isKeyBackupDownloadConfigured(): boolean {
-        return this.configuration !== null;
-    }
-
-    /**
-     * Return the details of the latest backup on the server, when we last checked.
-     *
-     * This is just a convenience method to expose {@link RustBackupManager.getServerBackupInfo}.
-     */
-    public async getServerBackupInfo(): Promise<KeyBackupInfo | null | undefined> {
-        return await this.backupManager.getServerBackupInfo();
-    }
-
-    /**
-     * Called when a MissingRoomKey or UnknownMessageIndex decryption error is encountered.
-     *
-     * This will try to download the key from the backup if there is a trusted active backup.
-     * In case of success the key will be imported and the onRoomKeysUpdated callback will be called
-     * internally by the rust-sdk and decryption will be retried.
-     *
-     * @param roomId - The room ID of the room where the error occurred.
-     * @param megolmSessionId - The megolm session ID that is missing.
-     */
-    public onDecryptionKeyMissingError(roomId: string, megolmSessionId: string): void {
-        // Several messages encrypted with the same session may be decrypted at the same time,
-        // so we need to be resistant and not query several time the same session.
-        if (this.isAlreadyInQueue(roomId, megolmSessionId)) {
-            // There is already a request queued for this session, no need to queue another one.
-            this.logger.trace(`Not checking key backup for session ${megolmSessionId} as it is already queued`);
-            return;
-        }
-
-        if (this.wasRequestedRecently(megolmSessionId)) {
-            // We already tried to download this session recently and it was not in backup, no need to try again.
-            this.logger.trace(
-                `Not checking key backup for session ${megolmSessionId} as it was already requested recently`,
-            );
-            return;
-        }
-
-        // We always add the request to the queue, even if we have a configuration problem (can't access backup).
-        // This is to make sure that if the configuration problem is resolved, we will try to download the key.
-        // This will happen after an initial sync, at this point the backup will not yet be trusted and the decryption
-        // key will not be available, but it will be just after the verification.
-        // We don't need to persist it because currently on refresh the sdk will retry to decrypt the messages in error.
-        this.queuedRequests.push({ roomId, megolmSessionId });
-
-        // Start the download loop if it's not already running.
-        this.downloadKeysLoop();
-    }
-
-    public stop(): void {
-        this.stopped = true;
-        this.backupManager.off(CryptoEvent.KeyBackupStatus, this.onBackupStatusChanged);
-        this.backupManager.off(CryptoEvent.KeyBackupFailed, this.onBackupStatusChanged);
-        this.backupManager.off(CryptoEvent.KeyBackupDecryptionKeyCached, this.onBackupStatusChanged);
-    }
-
-    /**
-     * Called when the backup status changes (CryptoEvents)
-     * This will trigger a check of the backup configuration.
-     */
-    private onBackupStatusChanged = (): void => {
-        // we want to force check configuration, so we clear the current one.
-        this.hasConfigurationProblem = false;
-        this.configuration = null;
-        this.getOrCreateBackupConfiguration().then((configuration) => {
-            if (configuration) {
-                // restart the download loop if it was stopped
-                this.downloadKeysLoop();
-            }
-        });
-    };
-
-    /** Returns true if the megolm session is already queued for download. */
-    private isAlreadyInQueue(roomId: string, megolmSessionId: string): boolean {
-        return this.queuedRequests.some((info) => {
-            return info.roomId == roomId && info.megolmSessionId == megolmSessionId;
-        });
-    }
-
-    /**
-     * Marks the session as not found in backup, to avoid retrying to soon for a key not in backup
-     *
-     * @param megolmSessionId - The megolm session ID that is missing.
-     */
-    private markAsNotFoundInBackup(megolmSessionId: string): void {
-        const now = Date.now();
-        this.sessionLastCheckAttemptedTime.set(megolmSessionId, now);
-        // if too big make some cleaning to keep under control
-        if (this.sessionLastCheckAttemptedTime.size > 100) {
-            this.sessionLastCheckAttemptedTime = new Map(
-                Array.from(this.sessionLastCheckAttemptedTime).filter((sid, ts) => {
-                    return Math.max(now - ts, 0) < KEY_BACKUP_BACKOFF;
-                }),
-            );
-        }
-    }
-
-    /** Returns true if the session was requested recently. */
-    private wasRequestedRecently(megolmSessionId: string): boolean {
-        const lastCheck = this.sessionLastCheckAttemptedTime.get(megolmSessionId);
-        if (!lastCheck) return false;
-        return Math.max(Date.now() - lastCheck, 0) < KEY_BACKUP_BACKOFF;
-    }
-
-    private async getBackupDecryptionKey(): Promise<RustSdkCryptoJs.BackupKeys | null> {
-        try {
-            return await this.olmMachine.getBackupKeys();
-        } catch {
-            return null;
-        }
-    }
-
-    /**
-     * Requests a key from the server side backup.
-     *
-     * @param version - The backup version to use.
-     * @param roomId - The room ID of the room where the error occurred.
-     * @param sessionId - The megolm session ID that is missing.
-     */
-    private async requestRoomKeyFromBackup(
-        version: string,
-        roomId: string,
-        sessionId: string,
-    ): Promise<KeyBackupSession> {
-        const path = encodeUri("/room_keys/keys/$roomId/$sessionId", {
-            $roomId: roomId,
-            $sessionId: sessionId,
-        });
-
-        return await this.http.authedRequest<KeyBackupSession>(Method.Get, path, { version }, undefined, {
-            prefix: ClientPrefix.V3,
-        });
-    }
-
-    private async downloadKeysLoop(): Promise<void> {
-        if (this.downloadLoopRunning) return;
-
-        // If we have a configuration problem, we don't want to try to download.
-        // If any configuration change is detected, we will retry and restart the loop.
-        if (this.hasConfigurationProblem) return;
-
-        this.downloadLoopRunning = true;
-
-        try {
-            while (this.queuedRequests.length > 0) {
-                // we just peek the first one without removing it, so if a new request for same key comes in while we're
-                // processing this one, it won't queue another request.
-                const request = this.queuedRequests[0];
-                try {
-                    // The backup could have changed between the time we queued the request and now, so we need to check
-                    const configuration = await this.getOrCreateBackupConfiguration();
-                    if (!configuration) {
-                        // Backup is not configured correctly, so stop the loop.
-                        this.downloadLoopRunning = false;
-                        return;
-                    }
-
-                    const result = await this.queryKeyBackup(request.roomId, request.megolmSessionId, configuration);
-
-                    if (this.stopped) {
-                        return;
-                    }
-                    // We got the encrypted key from backup, let's try to decrypt and import it.
-                    try {
-                        await this.decryptAndImport(request, result, configuration);
-                    } catch (e) {
-                        this.logger.error(
-                            `Error while decrypting and importing key backup for session ${request.megolmSessionId}`,
-                            e,
-                        );
-                    }
-                    // now remove the request from the queue as we've processed it.
-                    this.queuedRequests.shift();
-                } catch (err) {
-                    if (err instanceof KeyDownloadError) {
-                        switch (err.code) {
-                            case KeyDownloadErrorCode.MISSING_DECRYPTION_KEY:
-                                this.markAsNotFoundInBackup(request.megolmSessionId);
-                                // continue for next one
-                                this.queuedRequests.shift();
-                                break;
-                            case KeyDownloadErrorCode.NETWORK_ERROR:
-                                // We don't want to hammer if there is a problem, so wait a bit.
-                                await sleep(KEY_BACKUP_BACKOFF);
-                                break;
-                            case KeyDownloadErrorCode.STOPPED:
-                                // If the downloader was stopped, we don't want to retry.
-                                this.downloadLoopRunning = false;
-                                return;
-                        }
-                    } else if (err instanceof KeyDownloadRateLimitError) {
-                        // we want to retry after the backoff time
-                        await sleep(err.retryMillis);
-                    }
-                }
-            }
-        } finally {
-            // all pending request have been processed, we can stop the loop.
-            this.downloadLoopRunning = false;
-        }
-    }
-
-    /**
-     * Query the backup for a key.
-     *
-     * @param targetRoomId - ID of the room that the session is used in.
-     * @param targetSessionId - ID of the session for which to check backup.
-     * @param configuration - The backup configuration to use.
-     */
-    private async queryKeyBackup(
-        targetRoomId: string,
-        targetSessionId: string,
-        configuration: Configuration,
-    ): Promise<KeyBackupSession> {
-        this.logger.debug(`Checking key backup for session ${targetSessionId}`);
-        if (this.stopped) throw new KeyDownloadError(KeyDownloadErrorCode.STOPPED);
-        try {
-            const res = await this.requestRoomKeyFromBackup(configuration.backupVersion, targetRoomId, targetSessionId);
-            this.logger.debug(`Got key from backup for sessionId:${targetSessionId}`);
-            return res;
-        } catch (e) {
-            if (this.stopped) throw new KeyDownloadError(KeyDownloadErrorCode.STOPPED);
-
-            this.logger.info(`No luck requesting key backup for session ${targetSessionId}: ${e}`);
-            if (e instanceof MatrixError) {
-                const errCode = e.data.errcode;
-                if (errCode == "M_NOT_FOUND") {
-                    // Unfortunately the spec doesn't give us a way to differentiate between a missing key and a wrong version.
-                    // Synapse will return:
-                    //     - "error": "Unknown backup version" if the version is wrong.
-                    //     - "error": "No room_keys found" if the key is missing.
-                    // It's useful to know if the key is missing or if the version is wrong.
-                    // As it's not spec'ed, we fall back on considering the key is not in backup.
-                    // Notice that this request will be lost if instead the backup got out of sync (updated from other session).
-                    throw new KeyDownloadError(KeyDownloadErrorCode.MISSING_DECRYPTION_KEY);
-                }
-                if (e.isRateLimitError()) {
-                    let waitTime: number | undefined;
-                    try {
-                        waitTime = e.getRetryAfterMs() ?? undefined;
-                    } catch (error) {
-                        this.logger.warn("Error while retrieving a rate-limit retry delay", error);
-                    }
-                    if (waitTime && waitTime > 0) {
-                        this.logger.info(`Rate limited by server, waiting ${waitTime}ms`);
-                    }
-                    throw new KeyDownloadRateLimitError(waitTime ?? KEY_BACKUP_BACKOFF);
-                }
-            }
-            throw new KeyDownloadError(KeyDownloadErrorCode.NETWORK_ERROR);
-        }
-    }
-
-    private async decryptAndImport(
-        sessionInfo: SessionInfo,
-        data: KeyBackupSession,
-        configuration: Configuration,
-    ): Promise<void> {
-        const sessionsToImport: Record<string, KeyBackupSession> = { [sessionInfo.megolmSessionId]: data };
-
-        const keys = await configuration!.decryptor.decryptSessions(sessionsToImport);
-        for (const k of keys) {
-            k.room_id = sessionInfo.roomId;
-        }
-        await this.backupManager.importBackedUpRoomKeys(keys, configuration.backupVersion);
-    }
-
-    /**
-     * Gets the current backup configuration or create one if it doesn't exist.
-     *
-     * When a valid configuration is found it is cached and returned for subsequent calls.
-     * Otherwise, if a check is forced or a check has not yet been done, a new check is done.
-     *
-     * @returns The backup configuration to use or null if there is a configuration problem.
-     */
-    private async getOrCreateBackupConfiguration(): Promise<Configuration | null> {
-        if (this.configuration) {
-            return this.configuration;
-        }
-
-        // We already tried to check the configuration and it failed.
-        // We don't want to try again immediately, we will retry if a configuration change is detected.
-        if (this.hasConfigurationProblem) {
-            return null;
-        }
-
-        // This method can be called rapidly by several emitted CryptoEvent, so we need to make sure that we don't
-        // query the server several times.
-        if (this.currentBackupVersionCheck != null) {
-            this.logger.debug(`Already checking server version, use current promise`);
-            return await this.currentBackupVersionCheck;
-        }
-
-        this.currentBackupVersionCheck = this.internalCheckFromServer();
-        try {
-            return await this.currentBackupVersionCheck;
-        } finally {
-            this.currentBackupVersionCheck = null;
-        }
-    }
-
-    private async internalCheckFromServer(): Promise<Configuration | null> {
-        let currentServerVersion = null;
-        try {
-            currentServerVersion = await this.backupManager.getServerBackupInfo();
-        } catch (e) {
-            this.logger.debug(`Backup: error while checking server version: ${e}`);
-            this.hasConfigurationProblem = true;
-            return null;
-        }
-        this.logger.debug(`Got current backup version from server: ${currentServerVersion?.version}`);
-
-        if (currentServerVersion?.algorithm != "m.megolm_backup.v1.curve25519-aes-sha2") {
-            this.logger.info(`Unsupported algorithm ${currentServerVersion?.algorithm}`);
-            this.hasConfigurationProblem = true;
-            return null;
-        }
-
-        if (!currentServerVersion?.version) {
-            this.logger.info(`No current key backup`);
-            this.hasConfigurationProblem = true;
-            return null;
-        }
-
-        const activeVersion = await this.backupManager.getActiveBackupVersion();
-        if (activeVersion == null || currentServerVersion.version != activeVersion) {
-            // Either the current backup version on server side is not trusted, or it is out of sync with the active version on the client side.
-            this.logger.info(
-                `The current backup version on the server (${currentServerVersion.version}) is not trusted. Version we are currently backing up to: ${activeVersion}`,
-            );
-            this.hasConfigurationProblem = true;
-            return null;
-        }
-
-        const backupKeys = await this.getBackupDecryptionKey();
-        if (!backupKeys?.decryptionKey) {
-            this.logger.debug(`Not checking key backup for session (no decryption key)`);
-            this.hasConfigurationProblem = true;
-            return null;
-        }
-
-        if (activeVersion != backupKeys.backupVersion) {
-            this.logger.debug(
-                `Version for which we have a decryption key (${backupKeys.backupVersion}) doesn't match the version we are backing up to (${activeVersion})`,
-            );
-            this.hasConfigurationProblem = true;
-            return null;
-        }
-
-        const authData = currentServerVersion.auth_data as Curve25519AuthData;
-        if (authData.public_key != backupKeys.decryptionKey.megolmV1PublicKey.publicKeyBase64) {
-            this.logger.debug(`Key backup on server does not match our decryption key`);
-            this.hasConfigurationProblem = true;
-            return null;
-        }
-
-        const backupDecryptor = this.backupManager.createBackupDecryptor(backupKeys.decryptionKey);
-        this.hasConfigurationProblem = false;
-        this.configuration = {
-            decryptor: backupDecryptor,
-            backupVersion: activeVersion,
-        };
-        return this.configuration;
-    }
-}