Paths

Table of Contentst

Differential D14003

[native] remove `BackupKeys` and use `QRAuthBackupData`
Needs ReviewPublic
Actions

Authored by kamil on Fri, Nov 22, 2:52 AM.

Tags

None

Referenced Files

	F3366089: D14003.diff
	Mon, Nov 25, 9:30 AM

	Unknown Object (File)
	Fri, Nov 22, 6:45 AM

	Unknown Object (File)
	Fri, Nov 22, 6:45 AM

	Unknown Object (File)
	Fri, Nov 22, 6:42 AM

Subscribers

Details

Reviewers

bartek
tomek

Summary

Part of ENG-9656

BackupKeys can be misleading and confused with actual Use Keys; additionally, it also contains backupID. This was duplicated with the QRAuthBackupData type, which is the same type, but it's more suitable as it's going to be used only in the context of the Secondary Device Login protocol.

Additionally renaming retrieveBackupKeys -> getQRAuthBackupData

Depends on D14002

Test Plan

Tested creating backup and secondary device login

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Fri, Nov 22, 2:52 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptFri, Nov 22, 2:52 AM

kamil added a child revision: D14004: [backup-client][native] refactor restoring logic and deprecate `restoreBackup`.Fri, Nov 22, 2:56 AM

kamil added a parent revision: D13968: [native] avoid showing SIWE backup message screen for non-primary devices.Fri, Nov 22, 2:56 AM

Harbormaster completed remote builds in B32767: Diff 45943.Fri, Nov 22, 3:31 AM

kamil published this revision for review.Fri, Nov 22, 3:59 AM

kamil edited the summary of this revision. (Show Details)

kamil added inline comments.

native/native_rust_library/src/backup.rs
498–505	This is removed in the next diff

Revision Contents
Changeset List

Path

Size

lib/

components/

qr-auth-provider.react.js

4 lines

types/

backup-types.js

13 lines

tunnelbroker/

qr-code-auth-message-types.js

2 lines

native/

cpp/

CommonCpp/

NativeModules/

CommCoreModule.h

2 lines

CommCoreModule.cpp

2 lines

_generated/

commJSI-generated.cpp

native_rust_library/

src/

1 line

profile/

secondary-device-qr-code-scanner.react.js

2 lines

qr-code/

qr-code-utils.js

8 lines

schema/

CommCoreModuleSchema.js

6 lines

Diff 45943

lib/components/qr-auth-provider.react.js

Loading...

lib/types/backup-types.js

Loading...

lib/types/tunnelbroker/qr-code-auth-message-types.js

Loading...

native/cpp/CommonCpp/NativeModules/CommCoreModule.h

Loading...

native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp

Loading...

native/cpp/CommonCpp/_generated/commJSI-generated.cpp

Loading...

native/cpp/CommonCpp/_generated/commJSI.h

Loading...

native/native_rust_library/src/backup.rs

Loading...

native/profile/secondary-device-qr-code-scanner.react.js

Loading...

native/qr-code/qr-code-utils.js

Loading...

native/schema/CommCoreModuleSchema.js

Loading...