Page MenuHomePhabricator

[identity] Verify lastPrimarySignature for singleton lists
ClosedPublic

Authored by bartek on Sep 2 2024, 11:25 PM.
Tags
None
Referenced Files
F4357702: D13222.id43844.diff
Wed, Feb 26, 11:38 AM
F4357700: D13222.id43984.diff
Wed, Feb 26, 11:38 AM
F4357649: D13222.diff
Wed, Feb 26, 11:37 AM
Unknown Object (File)
Sat, Feb 15, 9:28 PM
Unknown Object (File)
Jan 15 2025, 6:56 AM
Unknown Object (File)
Dec 29 2024, 7:03 AM
Unknown Object (File)
Dec 29 2024, 7:03 AM
Unknown Object (File)
Dec 29 2024, 7:03 AM
Subscribers

Details

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
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Sep 3 2024, 12:04 AM
This revision is now accepted and ready to land.Sep 6 2024, 5:43 AM