HomePhabricator
Diffusion Comm 9ad37a92adf9

[protos] Add initial_device_list field

Description

[protos] Add initial_device_list field

Summary:
Addresses:

Added the initial_device_list to registration requests' proto messages.

We need it in the following rpcs:

  • RegisterPasswordUserStart
  • RegisterReservedPasswordUser
  • RegisterWalletUser
  • RegisterReservedWalletUser

The WalletAuthRequest is also used in LogInWalletUser so we always put empty string there

Test Plan:
Made sure the change is backwards compatible

  1. Compiled native without this changes
  2. Applied this change and compiled Identity Service. Added server-side log to print the initial_device_list.is_empty() value.
  3. Without recompiling, ran the registration on native.
  4. Made sure the value is an empty string
  5. Compiled native with changes from this diff, ran registration again and verified it's empty string

Reviewers: ashoat, will

Reviewed By: ashoat, will

Subscribers: tomek, varun

Differential Revision: https://phab.comm.dev/D12017

Details

Provenance
bartekAuthored on May 13 2024, 8:03 AM
Reviewer
ashoat
Differential Revision
D12017: [protos] Add initial_device_list field
Parents
rCOMMab54d73667e3: [web] Show alert when login fails due to outdated client version
Branches
Unknown
Tags
Unknown