Page MenuHomePhabricator

[identity] Add metadata parser for PlatformDetails
ClosedPublic

Authored by bartek on Jun 3 2024, 4:10 AM.
Tags
None
Referenced Files
F2902481: D12270.id40834.diff
Sat, Oct 5, 5:41 PM
F2902386: D12270.id40917.diff
Sat, Oct 5, 5:20 PM
F2899527: D12270.id41042.diff
Sat, Oct 5, 7:21 AM
Unknown Object (File)
Fri, Oct 4, 2:03 PM
Unknown Object (File)
Thu, Oct 3, 5:45 AM
Unknown Object (File)
Wed, Oct 2, 3:31 PM
Unknown Object (File)
Sat, Sep 28, 10:33 PM
Unknown Object (File)
Sat, Sep 14, 9:16 PM
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
Lint Not Applicable
Unit
Tests Not Applicable