- Remove deprecated `MatrixClient.keyBackupKeyFromPassword` call. (#28233)

- The `keyBackupKeyFromPassword` call is bound to the `keyCallback` props in `RestoreKeyBackupDialog` which is not used in `CreateSecretStorageDialog`
- Remove code bound to `keyCallback` props
pull/28254/head
Florian Duros 2024-10-21 11:59:07 +02:00 committed by GitHub
parent 3294c6b5c5
commit 061c8217b3
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 0 additions and 19 deletions

View File

@ -404,13 +404,10 @@ export default class CreateSecretStorageDialog extends React.PureComponent<IProp
}; };
private restoreBackup = async (): Promise<void> => { private restoreBackup = async (): Promise<void> => {
const keyCallback = (k: Uint8Array): void => {};
const { finished } = Modal.createDialog( const { finished } = Modal.createDialog(
RestoreKeyBackupDialog, RestoreKeyBackupDialog,
{ {
showSummary: false, showSummary: false,
keyCallback,
}, },
undefined, undefined,
/* priority = */ false, /* priority = */ false,

View File

@ -37,9 +37,6 @@ interface IProps {
// if false, will close the dialog as soon as the restore completes successfully // if false, will close the dialog as soon as the restore completes successfully
// default: true // default: true
showSummary?: boolean; showSummary?: boolean;
// If specified, gather the key from the user but then call the function with the backup
// key rather than actually (necessarily) restoring the backup.
keyCallback?: (key: Uint8Array) => void;
onFinished(done?: boolean): void; onFinished(done?: boolean): void;
} }
@ -156,13 +153,6 @@ export default class RestoreKeyBackupDialog extends React.PureComponent<IProps,
this.state.backupInfo, this.state.backupInfo,
{ progressCallback: this.progressCallback }, { progressCallback: this.progressCallback },
); );
if (this.props.keyCallback) {
const key = await MatrixClientPeg.safeGet().keyBackupKeyFromPassword(
this.state.passPhrase,
this.state.backupInfo,
);
this.props.keyCallback(key);
}
if (!this.props.showSummary) { if (!this.props.showSummary) {
this.props.onFinished(true); this.props.onFinished(true);
@ -197,10 +187,6 @@ export default class RestoreKeyBackupDialog extends React.PureComponent<IProps,
this.state.backupInfo, this.state.backupInfo,
{ progressCallback: this.progressCallback }, { progressCallback: this.progressCallback },
); );
if (this.props.keyCallback) {
const key = decodeRecoveryKey(this.state.recoveryKey);
this.props.keyCallback(key);
}
if (!this.props.showSummary) { if (!this.props.showSummary) {
this.props.onFinished(true); this.props.onFinished(true);
return; return;

View File

@ -122,7 +122,6 @@ describe("CreateSecretStorageDialog", () => {
expect(modalSpy).toHaveBeenCalledWith( expect(modalSpy).toHaveBeenCalledWith(
RestoreKeyBackupDialog, RestoreKeyBackupDialog,
{ {
keyCallback: expect.any(Function),
showSummary: false, showSummary: false,
}, },
undefined, undefined,
@ -200,7 +199,6 @@ describe("CreateSecretStorageDialog", () => {
expect(modalSpy).toHaveBeenCalledWith( expect(modalSpy).toHaveBeenCalledWith(
RestoreKeyBackupDialog, RestoreKeyBackupDialog,
{ {
keyCallback: expect.any(Function),
showSummary: false, showSummary: false,
}, },
undefined, undefined,