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
Unknown Object (File)
Tue, May 21, 8:26 PM
Unknown Object (File)
Mon, May 13, 12:33 AM
Unknown Object (File)
Apr 4 2024, 7:02 PM
Unknown Object (File)
Apr 4 2024, 6:43 PM
Unknown Object (File)
Mar 17 2024, 10:53 AM
Unknown Object (File)
Mar 11 2024, 8:14 AM
Unknown Object (File)
Mar 11 2024, 3:29 AM
Unknown Object (File)
Mar 10 2024, 2:38 PM
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
Lint Not Applicable
Unit
Tests Not Applicable