Page MenuHomePhabricator

[lib] Introduce SET_SESSION_RECOVERY_IN_PROGRESS action
ClosedPublic

Authored by ashoat on Feb 5 2024, 12:26 PM.
Tags
None
Referenced Files
F3382264: D10948.id36640.diff
Thu, Nov 28, 9:32 AM
F3381933: D10948.diff
Thu, Nov 28, 7:23 AM
Unknown Object (File)
Sun, Nov 24, 6:03 PM
Unknown Object (File)
Sun, Nov 24, 4:32 PM
Unknown Object (File)
Oct 29 2024, 10:48 AM
Unknown Object (File)
Oct 4 2024, 3:44 AM
Unknown Object (File)
Oct 4 2024, 3:44 AM
Unknown Object (File)
Oct 4 2024, 3:44 AM
Subscribers
None

Details

Summary

We'll use this action to start a keyserver session recovery. It will be dispatched from CallKeyserverEndpointProvider in a later diff.

This action will also be used to conclude a failed keyserver session recovery. However, a successful keyserver session recovery will be concluded with a SET_NEW_SESSION action.

Depends on D10947

Test Plan
  1. Test successful session invalidation in single keyserver world
    1. I prevented the Socket from rendering by adding a return null line before the other returns in KeyserverConnectionHandler. This avoided the Socket triggering session recovery.
    2. I started the iOS simulator and logged in using a test user.
    3. I opened the Redux Dev Tools
    4. I deleted the test user's cookie from the MariaDB database: DELETE FROM cookies WHERE user = 6390578 AND platform = 'ios'
    5. I sent a message as the test user
    6. I confirmed that session recovery was triggered in the Redux dev tools (and through some console logs)
    7. I repeated the process above several times to make sure it consistently worked multiple times in a single run
    8. I confirmed that the message was delivered "transparently" (without any visible issues, or evidence of session invalidation)
  2. Test failed session invalidation in single keyserver world
    1. I ran through the above test, but I hacked legacy-recover-keyserver-session.js to use the wrong password so the session recovery would fail
    2. I confirmed that I was logged out, and that an alert appeared explaining that my session was invalidated
  3. Test logging out during session recovery
    1. I triggered an infinite loop of session recoveries by running through the above test, but swallowing the SET_NEW_SESSION
    2. I logged out of the app
    3. I confirmed that the session recovery loop stopped, and that I was logged out successfully

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage