Paths

Table of Contentst

Differential D13222

[identity] Verify lastPrimarySignature for singleton lists
ClosedPublic
Actions

Authored by bartek on Mon, Sep 2, 11:25 PM.

Tags

None

Referenced Files

	F2711332: D13222.id.diff
	Sun, Sep 15, 10:48 PM

	F2711305: D13222.id43844.diff
	Sun, Sep 15, 10:36 PM

	F2708753: D13222.id43984.diff
	Sun, Sep 15, 4:37 PM

	F2708662: D13222.diff
	Sun, Sep 15, 3:34 PM

	F2707562: D13222.id43984.diff
	Sun, Sep 15, 12:09 PM

	F2706430: D13222.id43844.diff
	Sun, Sep 15, 7:53 AM

	Unknown Object (File)
	Sat, Sep 14, 6:28 PM

	Unknown Object (File)
	Sat, Sep 14, 1:08 PM

View All 18 Files

Subscribers

Details

Reviewers

varun
kamil

Commits

rCOMMa204eb8ceed3: [identity] Verify lastPrimarySignature for singleton lists

Summary

Required for ENG-8212.
In case of backup restore, the singleton device list will also have lastPrimarySignature, which wasn't accepted before for singleton device lists.

Test Plan

Existing tests still pass.
Tested manually with lastPrimarySignature being present:

Passes with no last primary signature provided and none expected (previous behavior unchanged)
Fails if signature provided but none expected
Fails no signature is provided but one is expected
Passes if valid signature is provided and it's expected (previous signing key provided)
Fails if signature verification fails for given signing public key

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Mon, Sep 2, 11:25 PM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMon, Sep 2, 11:25 PM

bartek added a child revision: D13223: [identity] Update SignedDeviceList::from_str to return tonic::Status.Mon, Sep 2, 11:27 PM

bartek mentioned this in D13224: [identity] Allow register_primary_device() to overwrite device list.Mon, Sep 2, 11:33 PM

Harbormaster completed remote builds in B31398: Diff 43844.Mon, Sep 2, 11:42 PM

bartek mentioned this in D13226: [identity] Implement RestoreUser RPCs.Mon, Sep 2, 11:43 PM

bartek published this revision for review.Tue, Sep 3, 12:04 AM

kamil accepted this revision.Fri, Sep 6, 5:43 AM

This revision is now accepted and ready to land.Fri, Sep 6, 5:43 AM

Closed by commit rCOMMa204eb8ceed3: [identity] Verify lastPrimarySignature for singleton lists (authored by bartek). · Explain WhyMon, Sep 9, 6:21 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMa204eb8ceed3: [identity] Verify lastPrimarySignature for singleton lists.

bartek mentioned this in rCOMMbee39ec012b7: [identity] Allow register_primary_device() to overwrite device list.

bartek mentioned this in rCOMM817c5955a206: [identity] Implement RestoreUser RPCs.Fri, Sep 13, 3:36 AM

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

44 lines

grpc_services/

authenticated.rs

1 line

1 line

Diff 43984

services/identity/src/device_list.rs

Loading...

services/identity/src/grpc_services/authenticated.rs

Loading...

services/identity/src/grpc_utils.rs

Loading...