HomePhabricator
Diffusion Comm aba103e8b24f

[native] Don't send reserved ETH addresses to registration flow

Description

[native] Don't send reserved ETH addresses to registration flow

Summary:
This fixes an issue I identified in ENG-8044, which is that we send ETH accounts that have been reserved with the primary keyserver to the registration flow. These accounts should not need to go through the registration flow; rather, they should be able to transparently login via the reserved usernames workflow.

The code change was inspired by the equivalent logic in ConnectEthereum here.

Note that after this diff, the login will still fail. ENG-4033 tracks updating the logic here (and in ExistingEthereumAccount) to try to register the account via the reserved usernames workflow (rather than a simple login).

I considered factoring this logic out, since both ConnectEthereum and FullscreenSIWEPanel need to check this. But in ENG-4033 we'll probably need to refactor that check anyways, since we'll want to behave differently in the case of a reserved username versus a fully-registered account.

Would normally put @varun on this, but as he's out I figured @inka would be a good reviewer.

Test Plan: Try logging in as comm.eth on staging. Confirm that it simply fails now, instead of redirecting the user to the registration flow

Reviewers: inka, varun

Reviewed By: varun

Subscribers: tomek, varun, inka

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

Details

Provenance
ashoatAuthored on May 5 2024, 8:59 PM
Reviewer
varun
Differential Revision
D11891: [native] Don't send reserved ETH addresses to registration flow
Parents
rCOMM4f223a5be331: [terraform] Fix lambda opensearch domain access
Branches
Unknown
Tags
Unknown