Page MenuHomePhabricator

[native] Fix new registration flow when not using CSAT
ClosedPublic

Authored by ashoat on May 7 2024, 10:49 AM.
Tags
None
Referenced Files
F3362275: D11926.id39902.diff
Sun, Nov 24, 9:53 PM
F3361209: D11926.id39904.diff
Sun, Nov 24, 3:47 PM
Unknown Object (File)
Thu, Nov 14, 9:44 AM
Unknown Object (File)
Thu, Nov 14, 5:45 AM
Unknown Object (File)
Mon, Nov 11, 10:58 AM
Unknown Object (File)
Mon, Nov 11, 10:18 AM
Unknown Object (File)
Mon, Nov 11, 7:51 AM
Unknown Object (File)
Sun, Nov 10, 10:11 PM
Subscribers
None

Details

Summary

This resolves ENG-8055. I broke this while working on ENG-7667.

When I extracted keyserverAuth into useRegistrationServerCall, I made that step a necessary part. I should have considered that we are still supporting !usingCommServicesAccessToken in the new registration, and that the keyserverAuth step should be skipped when !usingCommServicesAccessToken, since auth with the keyserver is handled in the first step.

Test Plan

First, I backed out of multi-keyserver mode by doing the following:

  1. Updating keyserver/secrets/user_credentials.json to set usingIdentityCredentials to false
  2. Removing keyserver/facts/authoritative_keyserver.json
  3. Removing native/facts/authoritative_keyserver.json
  4. Replacing keyserver/secrets/db_config.json so that it points to a DB initialized in a single-keyserver environment
  5. Deleted the Android app from my emulator and the iOS app from my simulator

I then tested the following scenarios:

  1. On Android, I tested registration as a password user using the legacy registration flow
  2. On Android, I tested registration as a password user using the new registration flow
  3. On Android, I tested login as a password user
  4. On iOS, I tested registration as an Ethereum user using the legacy registration flow (main SIWE button)
  5. On iOS, I tested registration as an Ethereum user using the new registration flow
  6. On iOS, I tested login as an Ethereum user (main SIWE button)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

native/account/registration/registration-server-call.js
286โ€“304 โ†—(On Diff #39902)

Flow won't let me combine these unfortunately

This revision is now accepted and ready to land.May 8 2024, 3:06 AM