Page MenuHomePhabricator

[native] Process username received in AuthResponse
ClosedPublic

Authored by bartek on May 20 2024, 11:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 2, 3:16 AM
Unknown Object (File)
Tue, Nov 26, 10:01 AM
Unknown Object (File)
Nov 13 2024, 8:31 AM
Unknown Object (File)
Nov 13 2024, 1:43 AM
Unknown Object (File)
Nov 11 2024, 10:51 AM
Unknown Object (File)
Nov 11 2024, 9:30 AM
Unknown Object (File)
Nov 10 2024, 10:07 PM
Unknown Object (File)
Nov 10 2024, 1:03 AM
Subscribers

Details

Summary

Part of ENG-8137.
Addresses ENG-7215 on native.

Modified native_rust_library and native Identity client to use username received in AuthResponse message from login/registration RPCs.

WARNING: We should first deploy Identity with D12127 before landing this diff. Otherwise, the Identity won't be sending username in responses and gRPC will default this to empty string, which potentially breaks all login / registration actions.

Depends on D12127

Test Plan

Console.log the return value of processAuthResult() in login and register RPCs (both password and wallet). Confirmed that username is present.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable