HomePhabricator
Diffusion Comm 0eb989146842

[native] Use unified keyserver session recovery in LoggedOutModal

Description

[native] Use unified keyserver session recovery in LoggedOutModal

Summary:
Instead of dispatching a call from the LoggedOutModal, we should use the unified mechanism owned by KeyserverConnectionHandler. This way, we can avoid multiple simultaneous recoveries being dispatched at once.

resetUserStateActionType will be deprecated in the next diff. It's okay to remove this because when we flip sessionRecoveryInProgress to true, one of two things will happen:

  1. The keyserver session recovery will conclude successfully and set a correct cookie and currentUserInfo.
  2. The keyserver session recovery will fail and trigger a session invalidation via SET_NEW_SESSION, which will result in both cookie and currentUserInfo being cleared.

Depends on D11212

Test Plan: Part 4.1 of this test plan: https://gist.github.com/Ashoat/75ab690d5c53cdd68a51b02e03e27c58

Reviewers: inka, tomek

Reviewed By: tomek

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

Details

Provenance
ashoatAuthored on Feb 12 2024, 9:24 PM
Reviewer
tomek
Differential Revision
D11213: [native] Use unified keyserver session recovery in LoggedOutModal
Parents
rCOMM7c9488ad1695: [lib] Replace sessionRecoveryInProgress with actionSessionRecovery
Branches
Unknown
Tags
Unknown