Page MenuHomePhabricator

[native] Use unified keyserver session recovery in LoggedOutModal
ClosedPublic

Authored by ashoat on Mar 3 2024, 8:46 PM.
Tags
None
Referenced Files
F2019925: D11213.diff
Sun, Jun 16, 2:00 PM
Unknown Object (File)
Thu, Jun 6, 11:17 PM
Unknown Object (File)
Thu, Jun 6, 5:38 PM
Unknown Object (File)
Thu, Jun 6, 12:46 AM
Unknown Object (File)
Wed, Jun 5, 10:40 PM
Unknown Object (File)
Wed, Jun 5, 10:24 PM
Unknown Object (File)
Tue, Jun 4, 12:04 PM
Unknown Object (File)
Tue, Jun 4, 5:33 AM
Subscribers
None

Details

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

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage