Paths

Table of Contentst

Differential D14002

[CommCoreModule] avoid calling Backup Service to get backup secrets
ClosedPublic
Actions

Authored by kamil on Nov 22 2024, 2:23 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Dec 17, 10:39 PM

	Unknown Object (File)
	Sat, Dec 14, 2:25 PM

	Unknown Object (File)
	Fri, Dec 13, 8:39 PM

	Unknown Object (File)
	Fri, Dec 13, 1:47 AM

	Unknown Object (File)
	Mon, Dec 9, 10:40 PM

	Unknown Object (File)
	Sat, Dec 7, 9:26 PM

	Unknown Object (File)
	Sat, Dec 7, 3:10 PM

	Unknown Object (File)
	Sat, Dec 7, 8:16 AM

View All 21 Files

Subscribers

Details

Reviewers

bartek
tomek

Commits

rCOMMf209543987d5: [CommCoreModule] avoid calling Backup Service to get backup secrets

Summary

Part of ENG-9656

It was identified in https://phab.comm.dev/D13849#387643.

We don't need to call the Backup Service, we have the data locally.

retrieveBackupKeys will be refactored later in the stack and used in the Restore protocol to fetch the User Keys from the backup service.

Depends on D14001

Test Plan

Tested creating backup and secondary device login

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Nov 22 2024, 2:23 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptNov 22 2024, 2:23 AM

rebase

kamil added a child revision: D14003: [native] remove `BackupKeys` and use `QRAuthBackupData`.Nov 22 2024, 2:52 AM

Harbormaster failed remote builds in B32763: Diff 45939!Nov 22 2024, 3:01 AM

Harbormaster completed remote builds in B32766: Diff 45942.Nov 22 2024, 3:22 AM

kamil edited the summary of this revision. (Show Details)Nov 22 2024, 3:56 AM

kamil added inline comments.Nov 22 2024, 3:59 AM

native/profile/secondary-device-qr-code-scanner.react.js
180–182 ↗	(On Diff #45942)	Previously we had validation because we received a string, which was downloaded from the Blob service and decrypted. Right now we have a JSI object so there is need to validate this.

kamil published this revision for review.Nov 22 2024, 4:00 AM

bartek accepted this revision.Tue, Nov 26, 1:30 AM

This revision is now accepted and ready to land.Tue, Nov 26, 1:30 AM

Closed by commit rCOMMf209543987d5: [CommCoreModule] avoid calling Backup Service to get backup secrets (authored by kamil). · Explain WhyTue, Dec 3, 6:28 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMf209543987d5: [CommCoreModule] avoid calling Backup Service to get backup secrets.

Revision Contents
Changeset List

Path

Size

native/

cpp/

CommonCpp/

NativeModules/

CommCoreModule.h

5 lines

CommCoreModule.cpp

68 lines

_generated/

commJSI-generated.cpp

profile/

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

24 lines

schema/

CommCoreModuleSchema.js

7 lines

Diff 46188

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/profile/secondary-device-qr-code-scanner.react.js

Loading...

native/schema/CommCoreModuleSchema.js

Loading...