Page MenuHomePhabricator

[identity] Add metadata parser for PlatformDetails
ClosedPublic

Authored by bartek on Jun 3 2024, 4:10 AM.
Tags
None
Referenced Files
F3243583: D12270.id40918.diff
Thu, Nov 14, 8:57 AM
Unknown Object (File)
Tue, Nov 12, 4:21 PM
Unknown Object (File)
Tue, Nov 12, 4:17 PM
Unknown Object (File)
Tue, Nov 12, 4:04 PM
Unknown Object (File)
Tue, Nov 12, 4:04 PM
Unknown Object (File)
Tue, Nov 12, 2:38 PM
Unknown Object (File)
Thu, Nov 7, 12:12 PM
Unknown Object (File)
Wed, Nov 6, 7:38 AM
Subscribers

Details

Summary

Reasoning in D12269 - we want to pass full PlatformDetails with Identity requests metadata, not only platform and code version.

Decided to introduce PlatformMetadata struct which accepts device type as string.
This is needed for custom device types (like "service") which are valid identity clients, but not valid devices for registration.
Parsing is handled in the next diff

Depends on D12269

Test Plan
  • Modified web client to introduce additional metadata in CodeVersionLayer.
  • Verified that calling get_platform_metadata() retrieve these params.
  • Verified that None is returned for state version and desktop version when not provided.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage