[lib] Clear sessionRecoveryInProgress during auth / deauth
Summary:
This is basically the login/logout case. When a keyserver is authorized or deauthorized, we should clear any active session recovery. These are the same cases when we clear the connectionIssue.
Depends on D10970
Test Plan:
- Test successful session invalidation in single keyserver world
- 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.
- I started the iOS simulator and logged in using a test user.
- I opened the Redux Dev Tools
- I deleted the test user's cookie from the MariaDB database: DELETE FROM cookies WHERE user = 6390578 AND platform = 'ios'
- I sent a message as the test user
- I confirmed that session recovery was triggered in the Redux dev tools (and through some console logs)
- I repeated the process above several times to make sure it consistently worked multiple times in a single run
- I confirmed that the message was delivered "transparently" (without any visible issues, or evidence of session invalidation)
- Test failed session invalidation in single keyserver world
- 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
- I confirmed that I was logged out, and that an alert appeared explaining that my session was invalidated
- Test logging out during session recovery
- I triggered an infinite loop of session recoveries by running through the above test, but swallowing the SET_NEW_SESSION
- I logged out of the app
- I confirmed that the session recovery loop stopped, and that I was logged out successfully
Reviewers: tomek, inka
Reviewed By: tomek
Differential Revision: https://phab.comm.dev/D10972